omxvideodec: Remove spurious unlock in error case
authorNicolas Dufresne <nicolas.dufresne@collabora.com>
Sat, 8 Sep 2018 02:55:41 +0000 (22:55 -0400)
committerNicolas Dufresne <nicolas.dufresne@collabora.com>
Mon, 10 Sep 2018 21:09:11 +0000 (17:09 -0400)
This was forgotton in previous patch. We no long hold the lock when goto
invalid_buffer is called.

https://bugzilla.gnome.org/show_bug.cgi?id=715192

omx/gstomxvideodec.c

index 848d0b1..e1aec3f 100644 (file)
@@ -1912,13 +1912,12 @@ eos:
 
     GST_VIDEO_DECODER_STREAM_LOCK (self);
     self->downstream_flow_ret = flow_ret;
+    GST_VIDEO_DECODER_STREAM_UNLOCK (self);
 
     /* Here we fallback and pause the task for the EOS case */
     if (flow_ret != GST_FLOW_OK)
       goto flow_error;
 
-    GST_VIDEO_DECODER_STREAM_UNLOCK (self);
-
     return;
   }
 
@@ -1958,7 +1957,6 @@ invalid_buffer:
         ("Invalid sized input buffer"));
     gst_pad_push_event (GST_VIDEO_DECODER_SRC_PAD (self), gst_event_new_eos ());
     gst_omx_video_dec_pause_loop (self, GST_FLOW_NOT_NEGOTIATED);
-    GST_VIDEO_DECODER_STREAM_UNLOCK (self);
     return;
   }