diff options
author | Zhao Zhili <quinkblack@foxmail.com> | 2022-02-05 00:19:35 +0800 |
---|---|---|
committer | Thilo Borgmann <thilo.borgmann@mail.de> | 2022-02-22 13:06:59 +0100 |
commit | 079de4991229fe18aeafee4ef2d420ce62861b81 (patch) | |
tree | e9283452f75442a9419400a3131f470cfae8f71e /libavdevice | |
parent | 6b708592fa8c80a8c65b063b25bb496920d1eda0 (diff) | |
download | ffmpeg-079de4991229fe18aeafee4ef2d420ce62861b81.tar.gz |
avdevice/avfoundation: check strdup
Diffstat (limited to 'libavdevice')
-rw-r--r-- | libavdevice/avfoundation.m | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/libavdevice/avfoundation.m b/libavdevice/avfoundation.m index 2078c4879c..8f5e2bd120 100644 --- a/libavdevice/avfoundation.m +++ b/libavdevice/avfoundation.m @@ -310,18 +310,21 @@ static void destroy_context(AVFContext* ctx) } } -static void parse_device_name(AVFormatContext *s) +static int parse_device_name(AVFormatContext *s) { AVFContext *ctx = (AVFContext*)s->priv_data; ctx->url = av_strdup(s->url); char *save; + if (!ctx->url) + return AVERROR(ENOMEM); if (ctx->url[0] != ':') { ctx->video_filename = av_strtok(ctx->url, ":", &save); ctx->audio_filename = av_strtok(NULL, ":", &save); } else { ctx->audio_filename = av_strtok(ctx->url, ":", &save); } + return 0; } /** @@ -760,6 +763,7 @@ static int get_audio_config(AVFormatContext *s) static int avf_read_header(AVFormatContext *s) { + int ret = 0; NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init]; uint32_t num_screens = 0; AVFContext *ctx = (AVFContext*)s->priv_data; @@ -812,7 +816,9 @@ static int avf_read_header(AVFormatContext *s) } // parse input filename for video and audio device - parse_device_name(s); + ret = parse_device_name(s); + if (ret) + goto fail; // check for device index given in filename if (ctx->video_device_index == -1 && ctx->video_filename) { @@ -1002,6 +1008,8 @@ static int avf_read_header(AVFormatContext *s) fail: [pool release]; destroy_context(ctx); + if (ret) + return ret; return AVERROR(EIO); } |