On Sun, Feb 11, 2024 at 1:03 AM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote:
> Connor Worley: > > - { > > - int w_block = avctx->coded_width / ctx->texture_block_w; > > - int h_block = avctx->coded_height / ctx->texture_block_h; > > - if (w_block * h_block * ctx->tex_step > ctx->tex_size * 8LL) > > - return AVERROR_INVALIDDATA; > > - } > > You removed this check without replacement. It presumably fixed a bug. > Did you test whether you reopened said bug? > (I think I already asked this in an earlier iteration of this patchset.) > > - Andreas > My change redefines tex_size to be equal to or less than the left-hand-side of that if statement, making it redundant AFAICT. I do see the check was added to fix 10979/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_DXV_fuzzer-6178582203203584, but I'm not familiar with the fuzzing system. If there's a way to replay the bad input, I'd be curious to try. -- Connor Worley _______________________________________________ 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".