From: Tim-Philipp Müller <tim@centricular.net>
Date: Mon, 22 Jan 2007 13:06:43 +0000 (+0000)
Subject: gst/avi/gstavidemux.c: Error out properly when pull_range fails while we're reading... 
X-Git-Tag: 1.19.3~509^2~12231
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a10f2478bbd1d04db5ceec6b0dc9eba6a6597bf8;p=platform%2Fupstream%2Fgstreamer.git

gst/avi/gstavidemux.c: Error out properly when pull_range fails while we're reading the headers, instead of just paus...

Original commit message from CVS:
* gst/avi/gstavidemux.c: (gst_avi_demux_stream_header_pull):
Error out properly when pull_range fails while we're reading the
headers, instead of just pausing the task silently. Fixes #399338.
---

diff --git a/ChangeLog b/ChangeLog
index d730e72..43b4a94 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2007-01-22  Tim-Philipp Müller  <tim at centricular dot net>
+
+	* gst/avi/gstavidemux.c: (gst_avi_demux_stream_header_pull):
+	  Error out properly when pull_range fails while we're reading the
+	  headers, instead of just pausing the task silently. Fixes #399338.
+
 2007-01-19  Tim-Philipp Müller  <tim at centricular dot net>
 
 	* gst/smpte/gstsmpte.c: (gst_smpte_collected):
diff --git a/gst/avi/gstavidemux.c b/gst/avi/gstavidemux.c
index f9c3597..6d23788 100644
--- a/gst/avi/gstavidemux.c
+++ b/gst/avi/gstavidemux.c
@@ -2639,7 +2639,7 @@ gst_avi_demux_stream_header_pull (GstAviDemux * avi)
   /* the header consists of a 'hdrl' LIST tag */
   res = gst_riff_read_chunk (element, avi->sinkpad, &avi->offset, &tag, &buf);
   if (res != GST_FLOW_OK)
-    return res;
+    goto pull_range_failed;
   else if (tag != GST_RIFF_TAG_LIST)
     goto no_list;
   else if (GST_BUFFER_SIZE (buf) < 4)
@@ -2658,7 +2658,7 @@ gst_avi_demux_stream_header_pull (GstAviDemux * avi)
     /* read new chunk */
     res = gst_riff_read_chunk (element, avi->sinkpad, &avi->offset, &tag, &buf);
     if (res != GST_FLOW_OK)
-      return res;
+      goto pull_range_failed;
     else if (tag != GST_RIFF_TAG_LIST)
       goto no_list;
     else if (GST_BUFFER_SIZE (buf) < 4)
@@ -2673,7 +2673,7 @@ gst_avi_demux_stream_header_pull (GstAviDemux * avi)
   else if (!gst_avi_demux_parse_avih (element, sub, &avi->avih))
     goto invalid_avih;
 
-  GST_DEBUG_OBJECT (avi, "AVI header ok, reading elemnts from header");
+  GST_DEBUG_OBJECT (avi, "AVI header ok, reading elements from header");
 
   /* now, read the elements from the header until the end */
   while (gst_riff_parse_chunk (element, buf, &offset, &tag, &sub)) {
@@ -2745,9 +2745,8 @@ gst_avi_demux_stream_header_pull (GstAviDemux * avi)
 
     res = gst_pad_pull_range (avi->sinkpad, avi->offset, 12, &buf);
     if (res != GST_FLOW_OK) {
-      GST_DEBUG_OBJECT (avi, "pull_ranged returned %s",
-          gst_flow_get_name (res));
-      return res;
+      GST_DEBUG_OBJECT (avi, "pull_range failure while looking for tags");
+      goto pull_range_failed;
     } else if (GST_BUFFER_SIZE (buf) < 12) {
       GST_DEBUG_OBJECT (avi, "got %d bytes which is less than 12 bytes",
           GST_BUFFER_SIZE (buf));
@@ -2769,9 +2768,8 @@ gst_avi_demux_stream_header_pull (GstAviDemux * avi)
               gst_riff_read_chunk (element, avi->sinkpad, &avi->offset, &tag,
               &buf);
           if (res != GST_FLOW_OK) {
-            GST_DEBUG_OBJECT (avi, "read_chunk returned %s",
-                gst_flow_get_name (res));
-            return res;
+            GST_DEBUG_OBJECT (avi, "couldn't read INFO chunk");
+            goto pull_range_failed;
           }
 
           sub = gst_buffer_create_sub (buf, 4, GST_BUFFER_SIZE (buf) - 4);
@@ -2893,6 +2891,12 @@ no_index:
         ("Could not get/create index"));
     return GST_FLOW_ERROR;
   }
+pull_range_failed:
+  {
+    GST_ELEMENT_ERROR (avi, STREAM, DEMUX, (NULL),
+        ("pull_range flow reading header: %s", gst_flow_get_name (res)));
+    return GST_FLOW_ERROR;
+  }
 }
 
 /* Do the actual seeking.