aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTimothy Gu <timothygu99@gmail.com>2016-12-05 09:24:47 -0800
committerMichael Niedermayer <michael@niedermayer.cc>2017-05-16 16:00:22 +0200
commit6522a5dcf09fe680342ee573fc1945ab622c7698 (patch)
treef18159efe7661a20a9178ac151b276bdc07cda46
parent987675ba0d942ff5cbb1a7c7486c0553006c6890 (diff)
downloadffmpeg-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.c4
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;
}