oggdemux: do not try to write empty header buffers
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Thu, 10 Nov 2011 14:18:54 +0000 (14:18 +0000)
committerVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Thu, 10 Nov 2011 14:38:09 +0000 (14:38 +0000)
Those are valid, and the EOS skeleton packet is actually empty.

ext/ogg/gstoggdemux.c

index e30fac7..08fa66a 100644 (file)
@@ -2290,7 +2290,8 @@ gst_ogg_demux_set_header_on_caps (GstOggDemux * ogg, GstCaps * caps,
     ogg_packet *op = headers->data;
     g_assert (op);
     buffer = gst_buffer_new_and_alloc (op->bytes);
-    gst_buffer_fill (buffer, 0, op->packet, op->bytes);
+    if (op->bytes)
+      gst_buffer_fill (buffer, 0, op->packet, op->bytes);
     GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_IN_CAPS);
     g_value_init (&value, GST_TYPE_BUFFER);
     gst_value_take_buffer (&value, buffer);