[FFmpeg-cvslog] au: move skipping of unused data to before parameter validation

[Justin Ruggles]

ffmpeg | branch: master | Justin Ruggles <justin.ruggles at gmail.com> | Sun Dec 23 13:33:33 2012 -0500| [c837b38dd33a11c3810e988a60193a858eb4f58c] | committer: Justin Ruggles

au: move skipping of unused data to before parameter validation

Also do not unnecessarily skip 0 bytes.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=c837b38dd33a11c3810e988a60193a858eb4f58c
---

 libavformat/au.c |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/libavformat/au.c b/libavformat/au.c
index c429ce1..f055a6f 100644
--- a/libavformat/au.c
+++ b/libavformat/au.c
@@ -81,6 +81,11 @@ static int au_read_header(AVFormatContext *s)
     rate = avio_rb32(pb);
     channels = avio_rb32(pb);
 
+    if (size > 24) {
+        /* skip unused data */
+        avio_skip(pb, size - 24);
+    }
+
     codec = ff_codec_get_id(codec_au_tags, id);
 
     if (codec == AV_CODEC_ID_NONE) {
@@ -99,11 +104,6 @@ static int au_read_header(AVFormatContext *s)
         return AVERROR_INVALIDDATA;
     }
 
-    if (size >= 24) {
-        /* skip unused data */
-        avio_skip(pb, size - 24);
-    }
-
     /* now we are ready: build format streams */
     st = avformat_new_stream(s, NULL);
     if (!st)