On 11/12/2022 8:44 PM, Michael Niedermayer wrote:
Fixes: Assertion failure
Fixes: 
51825/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MJPEGB_fuzzer-6393802688692224

Found-by: continuous fuzzing process 
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
---
  libavcodec/mjpegbdec.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavcodec/mjpegbdec.c b/libavcodec/mjpegbdec.c
index 98c64b44ca..a82a95d70a 100644
--- a/libavcodec/mjpegbdec.c
+++ b/libavcodec/mjpegbdec.c
@@ -141,9 +141,10 @@ read_header:
          av_log(avctx, AV_LOG_WARNING, "no picture\n");
          return buf_size;
      }
-
      av_frame_move_ref(rframe, s->picture_ptr);
      s->got_picture = 0;
+    if (avctx->skip_frame == AVDISCARD_ALL)
+        return AVERROR(EAGAIN);

Decode callback decoders don't return EAGAIN. If they don't generate a frame but there was no error, then they return the amount of bytes consumed while ensuring got_frame is 0.

      *got_frame = 1;
if (!s->lossless && avctx->debug & FF_DEBUG_QP) {
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to