summaryrefslogtreecommitdiff
path: root/libavcodec/omx.c
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2016-12-29 02:19:27 +0100
committerMichael Niedermayer <michael@niedermayer.cc>2017-01-08 15:25:14 +0100
commitbd83c295fc1b7f8001e5d134b912af86cd62c3f2 (patch)
treef83ddd8f8166d23e276e2b3bf2db3edb135e8b3d /libavcodec/omx.c
parent95d9a85ca3e662388d5fa7ef1937d1c3fbe2dcd5 (diff)
avcodec/omx: Do not pass negative value into av_malloc()
Fixes CID1396849 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec/omx.c')
-rw-r--r--libavcodec/omx.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/libavcodec/omx.c b/libavcodec/omx.c
index b5093f4941..16df50e456 100644
--- a/libavcodec/omx.c
+++ b/libavcodec/omx.c
@@ -761,7 +761,10 @@ static int omx_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
} else {
// If not, we need to allocate a new buffer with the right
// size and copy the input frame into it.
- uint8_t *buf = av_malloc(av_image_get_buffer_size(avctx->pix_fmt, s->stride, s->plane_size, 1));
+ uint8_t *buf = NULL;
+ int image_buffer_size = av_image_get_buffer_size(avctx->pix_fmt, s->stride, s->plane_size, 1);
+ if (image_buffer_size >= 0)
+ buf = av_malloc(image_buffer_size);
if (!buf) {
// Return the buffer to the queue so it's not lost
append_buffer(&s->input_mutex, &s->input_cond, &s->num_free_in_buffers, s->free_in_buffers, buffer);