diff options
author | Michael Niedermayer <michaelni@gmx.at> | 2015-02-18 13:40:39 +0100 |
---|---|---|
committer | Michael Niedermayer <michaelni@gmx.at> | 2015-02-18 13:40:39 +0100 |
commit | 9f6431c8f6c4e92e3f6ea2f3bc8f58677a7e7ce3 (patch) | |
tree | f30af41190b2911e3a9f36777321efc159b1640b | |
parent | bdb31942174c4673c7f212378951366e0704668d (diff) | |
download | ffmpeg-9f6431c8f6c4e92e3f6ea2f3bc8f58677a7e7ce3.tar.gz |
avfilter/af_channelmap: Move potential dereference after NULL check in get_channel_idx()
Fixes CID1270822
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r-- | libavfilter/af_channelmap.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/libavfilter/af_channelmap.c b/libavfilter/af_channelmap.c index 2a43c12af2..cc66f0436f 100644 --- a/libavfilter/af_channelmap.c +++ b/libavfilter/af_channelmap.c @@ -88,13 +88,14 @@ static char* split(char *message, char delim) { static int get_channel_idx(char **map, int *ch, char delim, int max_ch) { - char *next = split(*map, delim); + char *next; int len; int n = 0; - if (!next && delim == '-') - return AVERROR(EINVAL); if (!*map) return AVERROR(EINVAL); + next = split(*map, delim); + if (!next && delim == '-') + return AVERROR(EINVAL); len = strlen(*map); sscanf(*map, "%d%n", ch, &n); if (n != len) |