diff options
author | Paul B Mahol <onemda@gmail.com> | 2012-11-24 10:27:22 +0000 |
---|---|---|
committer | Paul B Mahol <onemda@gmail.com> | 2012-11-24 10:28:23 +0000 |
commit | ed591ed842dad0e23e7ba83d854faa61d2e2eb0f (patch) | |
tree | e33c1cd34ed780447c65eb6089fd95d8e85643a0 | |
parent | 1a61c7a54b7e93da0d3dcc4b19205f55db7da4d8 (diff) | |
download | ffmpeg-ed591ed842dad0e23e7ba83d854faa61d2e2eb0f.tar.gz |
rl2: return meaningful error codes
Signed-off-by: Paul B Mahol <onemda@gmail.com>
-rw-r--r-- | libavcodec/rl2.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/libavcodec/rl2.c b/libavcodec/rl2.c index 5cffd52901..c51fbd5873 100644 --- a/libavcodec/rl2.c +++ b/libavcodec/rl2.c @@ -140,7 +140,7 @@ static av_cold int rl2_decode_init(AVCodecContext *avctx) /** parse extra data */ if(!avctx->extradata || avctx->extradata_size < EXTRADATA1_SIZE){ av_log(avctx, AV_LOG_ERROR, "invalid extradata size\n"); - return -1; + return AVERROR_INVALIDDATA; } /** get frame_offset */ @@ -149,7 +149,7 @@ static av_cold int rl2_decode_init(AVCodecContext *avctx) if(s->video_base >= avctx->width * avctx->height){ av_log(avctx, AV_LOG_ERROR, "invalid video_base\n"); - return -1; + return AVERROR_INVALIDDATA; } /** initialize palette */ @@ -162,7 +162,7 @@ static av_cold int rl2_decode_init(AVCodecContext *avctx) if(back_size > 0){ unsigned char* back_frame = av_mallocz(avctx->width*avctx->height); if(!back_frame) - return -1; + return AVERROR(ENOMEM); rl2_rle_decode(s,avctx->extradata + EXTRADATA1_SIZE,back_size, back_frame,avctx->width,0); s->back_frame = back_frame; @@ -178,15 +178,16 @@ static int rl2_decode_frame(AVCodecContext *avctx, const uint8_t *buf = avpkt->data; int buf_size = avpkt->size; Rl2Context *s = avctx->priv_data; + int ret; if(s->frame.data[0]) avctx->release_buffer(avctx, &s->frame); /** get buffer */ s->frame.reference= 0; - if(avctx->get_buffer(avctx, &s->frame)) { + if ((ret = avctx->get_buffer(avctx, &s->frame)) < 0) { av_log(s->avctx, AV_LOG_ERROR, "get_buffer() failed\n"); - return -1; + return ret; } /** run length decode */ |