From c9c4aedaa67d9468cb952821a6ec0cdebdf109e8 Mon Sep 17 00:00:00 2001 From: Wim Taymans Date: Tue, 24 Jul 2012 14:36:58 +0200 Subject: [PATCH] oggdemux: don't copy empty packets 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 | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/ext/ogg/gstoggdemux.c b/ext/ogg/gstoggdemux.c index 17bf0e959b..ec22e4bd4a 100644 --- a/ext/ogg/gstoggdemux.c +++ b/ext/ogg/gstoggdemux.c @@ -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;