[FFmpeg-cvslog] omx: Fix allocation check
Timothy Gu
git at videolan.org
Fri Apr 14 00:32:29 EEST 2017
ffmpeg | branch: master | Timothy Gu <timothygu99 at gmail.com> | Mon Dec 5 09:29:12 2016 -0800| [d3da8a0035734529c4e26696c9a0c6cb56633838] | committer: Vittorio Giovara
omx: Fix allocation check
Also use av_mallocz_array().
Bug-Id: CID 1396839
Signed-off-by: Vittorio Giovara <vittorio.giovara at gmail.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d3da8a0035734529c4e26696c9a0c6cb56633838
---
libavcodec/omx.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/omx.c b/libavcodec/omx.c
index 0c61c2f536..05c874323c 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;
}
More information about the ffmpeg-cvslog
mailing list