From 2ba6d7cb8278970bb9971448706c1e44ba043a81 Mon Sep 17 00:00:00 2001 From: Pan Bian Date: Mon, 27 Nov 2017 09:12:54 +0800 Subject: ffmpeg: Check read_ffserver_streams() return value The function avformat_alloc_context() will return a NULL pointer on failure. However, in function read_ffserver_streams(), its return value is not validated and the subsequent dereference may result in a bad memory access bug. Check its return value against NULL and avoid potential NULL dereference. Signed-off-by: Pan Bian Signed-off-by: Michael Niedermayer --- fftools/ffmpeg_opt.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'fftools/ffmpeg_opt.c') diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index 9445a2dd11..672054223a 100644 --- a/fftools/ffmpeg_opt.c +++ b/fftools/ffmpeg_opt.c @@ -2000,6 +2000,8 @@ static int read_ffserver_streams(OptionsContext *o, AVFormatContext *s, const ch { int i, err; AVFormatContext *ic = avformat_alloc_context(); + if (!ic) + return AVERROR(ENOMEM); ic->interrupt_callback = int_cb; err = avformat_open_input(&ic, filename, NULL, NULL); -- cgit v1.2.3