[FFmpeg-devel] [PATCH] avdevice/lavfi: fix crash on unconnected outpads
metamuffin
metamuffin at disroot.org
Thu Jun 1 13:13:36 EEST 2023
Nameless outpads would cause an invocation to sscanf with NULL.
Example: ffmpeg -f lavfi -i 'nullsrc;nullsrc' -
Changed to throwing an error instead.
---
libavdevice/lavfi.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
index 9c1fcf334b..21f33ade0e 100644
--- a/libavdevice/lavfi.c
+++ b/libavdevice/lavfi.c
@@ -174,6 +174,11 @@ av_cold static int lavfi_read_header(AVFormatContext *avctx)
* create a mapping between them and the streams */
for (i = 0, inout = output_links; inout; i++, inout = inout->next) {
int stream_idx = 0, suffix = 0, use_subcc = 0;
+ if (!inout->name) {
+ av_log(avctx, AV_LOG_ERROR,
+ "Filtergraph has nameless outpads\n");
+ FAIL(AVERROR(EINVAL));
+ }
sscanf(inout->name, "out%n%d%n", &suffix, &stream_idx, &suffix);
if (!suffix) {
av_log(avctx, AV_LOG_ERROR,
--
2.40.1
More information about the ffmpeg-devel
mailing list