From f6d8dee68de0e4a8a035c74a75d43a6494a00804 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tim-Philipp=20M=C3=BCller?= <tim@centricular.net>
Date: Tue, 9 Aug 2005 14:40:02 +0000
Subject: [PATCH] gst/avi/gstavidemux.c: Fix AVI header parsing: add missing
 break statement after

Original commit message from CVS:
* gst/avi/gstavidemux.c: (gst_avi_demux_stream_header):
Fix AVI header parsing: add missing break statement after
GST_RIFF_INFO_LIST parsing code; gst_riff_read_chunk() has
already advanced the avi->offset, no need to do it twice
(fixes MovieOfMovies.avi).
---
 ChangeLog             | 8 ++++++++
 gst/avi/gstavidemux.c | 4 ++++
 2 files changed, 12 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index 51f2aec489..7a8b197838 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2005-08-09  Tim-Philipp Müller  <tim at centricular dot net>
+
+	* gst/avi/gstavidemux.c: (gst_avi_demux_stream_header):
+	  Fix AVI header parsing: add missing break statement after
+	  GST_RIFF_INFO_LIST parsing code; gst_riff_read_chunk() has
+	  already advanced the avi->offset, no need to do it twice
+	  (fixes MovieOfMovies.avi).
+
 2005-08-09  Tim-Philipp Müller  <tim at centricular dot net>
 
 	* ext/jpeg/gstjpegdec.c: (gst_jpeg_dec_init),
diff --git a/gst/avi/gstavidemux.c b/gst/avi/gstavidemux.c
index fcf28c3447..40ad6f8a99 100644
--- a/gst/avi/gstavidemux.c
+++ b/gst/avi/gstavidemux.c
@@ -1848,6 +1848,8 @@ gst_avi_demux_stream_header (GstAviDemux * avi)
         avi->avih->streams, avi->num_streams);
   }
 
+  GST_DEBUG_OBJECT (avi, "skipping junk between header and data ...");
+
   /* Now, find the data (i.e. skip all junk between header and data) */
   do {
     guint size;
@@ -1881,6 +1883,8 @@ gst_avi_demux_stream_header (GstAviDemux * avi)
               gst_tag_list_free (t);
             gst_buffer_unref (buf);
           }
+          /* gst_riff_read_chunk() has already advanced avi->offset */
+          break;
         default:
           avi->offset += 8 + ((size + 1) & ~1);
           break;