nuv: check per-frame header for validity.
authorReimar Döffinger <Reimar.Doeffinger@gmx.de>
Sat, 5 Nov 2011 22:13:07 +0000 (23:13 +0100)
committerAnton Khirnov <anton@khirnov.net>
Sat, 26 Nov 2011 07:38:59 +0000 (08:38 +0100)
Since it contains dimensions parsing an invalid one has rather
annoying effects.

Signed-off-by: Anton Khirnov <anton@khirnov.net>
libavcodec/nuv.c

index 7bace2e..94962b5 100644 (file)
@@ -184,9 +184,9 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *data_size,
     }
     if (c->codec_frameheader) {
         int w, h, q;
-        if (buf_size < 12) {
-            av_log(avctx, AV_LOG_ERROR, "invalid nuv video frame\n");
-            return -1;
+        if (buf[0] != 'V' || buf_size < 12) {
+            av_log(avctx, AV_LOG_ERROR, "invalid nuv video frame (wrong codec_tag?)\n");
+            return AVERROR_INVALIDDATA;
         }
         w = AV_RL16(&buf[6]);
         h = AV_RL16(&buf[8]);