diff options
author | Timothy Gu <timothygu99@gmail.com> | 2016-12-05 09:24:47 -0800 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2017-05-16 16:00:22 +0200 |
commit | 6522a5dcf09fe680342ee573fc1945ab622c7698 (patch) | |
tree | f18159efe7661a20a9178ac151b276bdc07cda46 | |
parent | 987675ba0d942ff5cbb1a7c7486c0553006c6890 (diff) | |
download | ffmpeg-6522a5dcf09fe680342ee573fc1945ab622c7698.tar.gz |
omx: Fix OOM check
Also use av_mallocz_array().
Fixes CID1396839.
(cherry picked from commit 16a75304fe42d3a007c78126b6370c94ccf891f6)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r-- | libavcodec/omx.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libavcodec/omx.c b/libavcodec/omx.c index 375dd1969c..1dabbbbf3f 100644 --- a/libavcodec/omx.c +++ b/libavcodec/omx.c @@ -352,12 +352,12 @@ static av_cold int find_component(OMXContext *omx_context, void *logctx, av_log(logctx, AV_LOG_WARNING, "No component for role %s found\n", role); return AVERROR_ENCODER_NOT_FOUND; } - components = av_mallocz(sizeof(char*) * num); + components = av_mallocz_array(num, sizeof(*components)); if (!components) return AVERROR(ENOMEM); for (i = 0; i < num; i++) { components[i] = av_mallocz(OMX_MAX_STRINGNAME_SIZE); - if (!components) { + if (!components[i]) { ret = AVERROR(ENOMEM); goto end; } |