diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2019-11-02 14:14:44 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2019-12-06 20:30:58 +0100 |
| commit | c005d08103f513b31c27819e25285a3840c028f4 (patch) | |
| tree | a2fe992df42b5ce5e3b2f253b8abaf040c593dee | |
| parent | 22b24fe8ae76d0ce3bc3cdbde37c4057cf59801a (diff) | |
| download | ffmpeg-c005d08103f513b31c27819e25285a3840c028f4.tar.gz | |
avcodec/nuv: Move comptype check up
Fixes: Timeout (23sec -> 5ms)
Fixes: 18517/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_NUV_fuzzer-5753135536013312
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 1138cdecbe0164ab1f07768418e794fddfdc636d)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavcodec/nuv.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/libavcodec/nuv.c b/libavcodec/nuv.c index 5294167034..d295707c6f 100644 --- a/libavcodec/nuv.c +++ b/libavcodec/nuv.c @@ -214,6 +214,14 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, case NUV_RTJPEG: minsize = c->width/16 * (c->height/16) * 6; break; + case NUV_BLACK: + case NUV_COPY_LAST: + case NUV_LZO: + case NUV_RTJPEG_IN_LZO: + break; + default: + av_log(avctx, AV_LOG_ERROR, "unknown compression\n"); + return AVERROR_INVALIDDATA; } if (buf_size < minsize / 4) return AVERROR_INVALIDDATA; @@ -302,9 +310,6 @@ retry: case NUV_COPY_LAST: /* nothing more to do here */ break; - default: - av_log(avctx, AV_LOG_ERROR, "unknown compression\n"); - return AVERROR_INVALIDDATA; } if ((result = av_frame_ref(picture, c->pic)) < 0) |