diff options
author | Stefano Sabatini <stefano.sabatini-lala@poste.it> | 2010-10-18 13:19:53 +0000 |
---|---|---|
committer | Stefano Sabatini <stefano.sabatini-lala@poste.it> | 2010-10-18 13:19:53 +0000 |
commit | 082a857522fb6b2cf1dc5f2d7251063c9531d62d (patch) | |
tree | e1971ff96c2fdf10b62668d3c442da4f4758fef8 /libavcodec/xan.c | |
parent | a3a29c265147de35413f2a25857a65e38398fa49 (diff) | |
download | ffmpeg-082a857522fb6b2cf1dc5f2d7251063c9531d62d.tar.gz |
Make XAN decoder return meaningful error codes.
Originally committed as revision 25516 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'libavcodec/xan.c')
-rw-r--r-- | libavcodec/xan.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/libavcodec/xan.c b/libavcodec/xan.c index 3f6aa8cee5..ebc2b420f0 100644 --- a/libavcodec/xan.c +++ b/libavcodec/xan.c @@ -69,7 +69,7 @@ static av_cold int xan_decode_init(AVCodecContext *avctx) if ((avctx->codec->id == CODEC_ID_XAN_WC3) && (s->avctx->palctrl == NULL)) { av_log(avctx, AV_LOG_ERROR, " WC3 Xan video: palette expected.\n"); - return -1; + return AVERROR(EINVAL); } avctx->pix_fmt = PIX_FMT_PAL8; @@ -77,12 +77,12 @@ static av_cold int xan_decode_init(AVCodecContext *avctx) s->buffer1_size = avctx->width * avctx->height; s->buffer1 = av_malloc(s->buffer1_size); if (!s->buffer1) - return -1; + return AVERROR(ENOMEM); s->buffer2_size = avctx->width * avctx->height; s->buffer2 = av_malloc(s->buffer2_size + 130); if (!s->buffer2) { av_freep(&s->buffer1); - return -1; + return AVERROR(ENOMEM); } return 0; @@ -359,13 +359,13 @@ static int xan_decode_frame(AVCodecContext *avctx, AVPacket *avpkt) { const uint8_t *buf = avpkt->data; - int buf_size = avpkt->size; + int ret, buf_size = avpkt->size; XanContext *s = avctx->priv_data; AVPaletteControl *palette_control = avctx->palctrl; - if (avctx->get_buffer(avctx, &s->current_frame)) { + if ((ret = avctx->get_buffer(avctx, &s->current_frame))) { av_log(s->avctx, AV_LOG_ERROR, " Xan Video: get_buffer() failed\n"); - return -1; + return ret; } s->current_frame.reference = 3; |