[FFmpeg-devel] [PATCH v2 1/2] fftools: do not access out of bounds filtergraph

Marvin Scholz epirat07 at gmail.com
Tue Oct 1 03:57:11 EEST 2024


The log message was logged for `filtergraphs[j]` which would cause a
heap buffer overflow in certain circumstances.

Correctly it should be logged for the current filtergraph, so just
use `fg` here.
---
 fftools/ffmpeg_filter.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c
index 2ff7c4c4f4..ddcd18dd15 100644
--- a/fftools/ffmpeg_filter.c
+++ b/fftools/ffmpeg_filter.c
@@ -1409,7 +1409,7 @@ int fg_finalise_bindings(void)
         for (int j = 0; j < fg->nb_outputs; j++) {
             OutputFilter *output = fg->outputs[j];
             if (!output->bound) {
-                av_log(filtergraphs[j], AV_LOG_FATAL,
+                av_log(fg, AV_LOG_FATAL,
                        "Filter %s has an unconnected output\n", output->name);
                 return AVERROR(EINVAL);
             }

base-commit: 358fdf30838682f2b183e67d247e0d4d53b5a6a4
-- 
2.39.3 (Apple Git-146)




More information about the ffmpeg-devel mailing list