oggdemux: don't copy empty packets
authorWim Taymans <wim.taymans@collabora.co.uk>
Tue, 24 Jul 2012 12:36:58 +0000 (14:36 +0200)
committerWim Taymans <wim.taymans@collabora.co.uk>
Tue, 24 Jul 2012 12:36:58 +0000 (14:36 +0200)
Don't try to copy empty packets, which contain a repeated frame in theora.

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

ext/ogg/gstoggdemux.c

index 17bf0e9..ec22e4b 100644 (file)
@@ -640,9 +640,11 @@ gst_ogg_demux_chain_peer (GstOggPad * pad, ogg_packet * packet,
   if (delta_unit)
     GST_BUFFER_FLAG_SET (buf, GST_BUFFER_FLAG_DELTA_UNIT);
 
-  /* copy packet in buffer */
-  gst_buffer_fill (buf, 0, packet->packet + offset,
-      packet->bytes - offset - trim);
+  if (packet->packet != NULL) {
+    /* copy packet in buffer */
+    gst_buffer_fill (buf, 0, packet->packet + offset,
+        packet->bytes - offset - trim);
+  }
 
   GST_BUFFER_TIMESTAMP (buf) = out_timestamp;
   GST_BUFFER_DURATION (buf) = out_duration;