oss4: fix a missing unlock and a return-only-when-assertions-enabled
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Fri, 2 May 2014 08:33:51 +0000 (09:33 +0100)
committerVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Fri, 2 May 2014 08:33:51 +0000 (09:33 +0100)
Spotted on the side while looking at another issue.

sys/oss4/oss4-property-probe.c

index 2e164f2..51ff3c1 100644 (file)
@@ -315,7 +315,8 @@ gst_oss4_property_probe_get_values (GstObject * probe, const gchar * pname)
     fd = GST_OSS4_SOURCE (probe)->fd;
   } else {
     GST_OBJECT_UNLOCK (obj);
-    g_return_val_if_reached (NULL);
+    g_assert_not_reached ();
+    return NULL;
   }
 
   /* copy fd if it's open, so we can just unconditionally close() later */
@@ -342,6 +343,7 @@ gst_oss4_property_probe_get_values (GstObject * probe, const gchar * pname)
   }
 
   if (devices == NULL) {
+    GST_OBJECT_UNLOCK (obj);
     GST_DEBUG_OBJECT (obj, "No devices found");
     goto done;
   }