From a9f3e4b138fc14f7512fde821c051fe1ff7f124f Mon Sep 17 00:00:00 2001 From: Marton Balint Date: Thu, 9 Feb 2017 00:03:21 +0100 Subject: [PATCH] avfilter/f_setcmd: fix null pointer dereference on using dash as interval Fixes Coverity CID 1396259. Reviewed-by: Paul B Mahol Signed-off-by: Marton Balint --- libavfilter/f_sendcmd.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/libavfilter/f_sendcmd.c b/libavfilter/f_sendcmd.c index fb30220e7c..522d6adb90 100644 --- a/libavfilter/f_sendcmd.c +++ b/libavfilter/f_sendcmd.c @@ -268,6 +268,13 @@ static int parse_interval(Interval *interval, int interval_count, char *start, *end; start = av_strtok(intervalstr, "-", &end); + if (!start) { + ret = AVERROR(EINVAL); + av_log(log_ctx, AV_LOG_ERROR, + "Invalid interval specification '%s' in interval #%d\n", + intervalstr, interval_count); + goto end; + } if ((ret = av_parse_time(&interval->start_ts, start, 1)) < 0) { av_log(log_ctx, AV_LOG_ERROR, "Invalid start time specification '%s' in interval #%d\n", -- 2.11.0