[Spice-devel] [PATCH spice-gtk v2 14/15] gstaudio: set state to NULL before unref

[Snir Sheriber]

On 1/9/19 12:09 PM, Frediano Ziglio wrote:
> From: Marc-André Lureau <marcandre.lureau at redhat.com>
>
> (lt-spicy:13097): GStreamer-CRITICAL **: 18:01:13.698:
> Trying to dispose element pipeline1, but it is in READY instead of the NULL state.
> You need to explicitly set elements to the NULL state before
> dropping the final reference, to allow them to clean up.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau at redhat.com>
> ---
>   src/spice-gstaudio.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/spice-gstaudio.c b/src/spice-gstaudio.c
> index 40050007..9ba321bb 100644
> --- a/src/spice-gstaudio.c
> +++ b/src/spice-gstaudio.c
> @@ -191,7 +191,7 @@ static void record_start(SpiceRecordChannel *channel, gint format, gint channels
>       if (p->record.pipe &&
>           (p->record.rate != frequency ||
>            p->record.channels != channels)) {
> -        record_stop(gstaudio);
> +        gst_element_set_state(p->record.pipe, GST_STATE_NULL);
>           g_clear_pointer(&p->record.pipe, gst_object_unref);
>       }
>   

Hi,

looks fine to me

Acked-by: Snir Sheriber <ssheribe at redhat.com>