aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2014-04-06 04:01:24 +0200
committerMichael Niedermayer <michaelni@gmx.at>2014-06-23 15:10:07 +0200
commit9c83914a1fd6bb9d19d5d81a4c25264176c2c25c (patch)
treea29c0d1b1bbf7eafca07eba3b2af2be1b0c01ee1
parent861732246ff26186cbaa12edfcdb40abd2fd81b1 (diff)
downloadffmpeg-9c83914a1fd6bb9d19d5d81a4c25264176c2c25c.tar.gz
avcodec/msrle: check return code for success before use
The check is possibly redundant, but better to check for errors that dont occur than to skip the check and crash Fixes CID1197060 Signed-off-by: Michael Niedermayer <michaelni@gmx.at> (cherry picked from commit 754f84663e8b3a88fa2e953b195d59230393fb8d) Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavcodec/msrle.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/libavcodec/msrle.c b/libavcodec/msrle.c
index 750e54ae99..9f0cac61e3 100644
--- a/libavcodec/msrle.c
+++ b/libavcodec/msrle.c
@@ -116,6 +116,9 @@ static int msrle_decode_frame(AVCodecContext *avctx,
uint8_t *buf = avpkt->data + (avctx->height-1)*istride;
int i, j;
+ if (linesize < 0)
+ return linesize;
+
for (i = 0; i < avctx->height; i++) {
if (avctx->bits_per_coded_sample == 4) {
for (j = 0; j < avctx->width - 1; j += 2) {