qtdemux: Only set width/height in caps if they're non-0
authorOlivier CrĂȘte <olivier.crete@collabora.com>
Thu, 26 Jul 2018 19:25:06 +0000 (15:25 -0400)
committerEdward Hervey <bilboed@bilboed.com>
Fri, 19 Oct 2018 12:19:27 +0000 (14:19 +0200)
If they are not valid, then let a downstream parser complete them.

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

gst/isomp4/qtdemux.c

index 74d8ca6..bf890dc 100644 (file)
@@ -8314,9 +8314,10 @@ gst_qtdemux_configure_stream (GstQTDemux * qtdemux, QtDemuxStream * stream)
       CUR_STREAM (stream)->caps =
           gst_caps_make_writable (CUR_STREAM (stream)->caps);
 
-      gst_caps_set_simple (CUR_STREAM (stream)->caps,
-          "width", G_TYPE_INT, CUR_STREAM (stream)->width,
-          "height", G_TYPE_INT, CUR_STREAM (stream)->height, NULL);
+      if (CUR_STREAM (stream)->width && CUR_STREAM (stream)->height)
+        gst_caps_set_simple (CUR_STREAM (stream)->caps,
+            "width", G_TYPE_INT, CUR_STREAM (stream)->width,
+            "height", G_TYPE_INT, CUR_STREAM (stream)->height, NULL);
 
       /* set framerate if calculated framerate is reliable */
       if (fps_available) {