[FFmpeg-cvslog] [ffmpeg] branch master updated. c9e93df4ee avcodec/prores_raw: Fix heap buffer overflow
ffmpeg-git at ffmpeg.org
ffmpeg-git at ffmpeg.org
Fri Aug 15 18:07:44 EEST 2025
The branch, master has been updated
via c9e93df4eed93fe0044c52d953688c4180de1d48 (commit)
from ac5303a5b9cdc17e4ebddfdc3203a4c325d42c52 (commit)
- Log -----------------------------------------------------------------
commit c9e93df4eed93fe0044c52d953688c4180de1d48
Author: Oliver Chang <ochang at google.com>
AuthorDate: Thu Aug 14 22:11:41 2025 -0700
Commit: Lynne <dev at lynne.ee>
CommitDate: Fri Aug 15 15:06:58 2025 +0000
avcodec/prores_raw: Fix heap buffer overflow
When dimensions differ from context, those were updated using
ff_set_dimensions, however this overwrote the aligned coded_width and
coded_height that were set before, leading to a buffer overflow when
writing the frame data.
Fixes: OssFuzz 438771336
Fixes: Heap-buffer-overflow
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Marvin Scholz <epirat07 at gmail.com>
Reviewed-by: Marvin Scholz <epirat07 at gmail.com>
diff --git a/libavcodec/prores_raw.c b/libavcodec/prores_raw.c
index 748e176815..b2aa97ddda 100644
--- a/libavcodec/prores_raw.c
+++ b/libavcodec/prores_raw.c
@@ -367,9 +367,6 @@ static int decode_frame(AVCodecContext *avctx,
if ((w & 1) || (h & 1))
return AVERROR_INVALIDDATA;
- avctx->coded_width = FFALIGN(w, 16);
- avctx->coded_height = FFALIGN(h, 16);
-
if (w != avctx->width || h != avctx->height) {
av_log(avctx, AV_LOG_WARNING, "picture resolution change: %ix%i -> %ix%i\n",
avctx->width, avctx->height, w, h);
@@ -377,6 +374,9 @@ static int decode_frame(AVCodecContext *avctx,
return ret;
}
+ avctx->coded_width = FFALIGN(w, 16);
+ avctx->coded_height = FFALIGN(h, 16);
+
enum AVPixelFormat pix_fmt = AV_PIX_FMT_BAYER_RGGB16;
if (pix_fmt != s->pix_fmt) {
s->pix_fmt = pix_fmt;
-----------------------------------------------------------------------
Summary of changes:
libavcodec/prores_raw.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
hooks/post-receive
--
More information about the ffmpeg-cvslog
mailing list