Make XAN decoder return meaningful error codes.

Originally committed as revision 25516 to svn://svn.ffmpeg.org/ffmpeg/trunk
oldabi
Stefano Sabatini 15 years ago
parent a3a29c2651
commit 082a857522
  1. 12
      libavcodec/xan.c

@ -69,7 +69,7 @@ static av_cold int xan_decode_init(AVCodecContext *avctx)
if ((avctx->codec->id == CODEC_ID_XAN_WC3) && if ((avctx->codec->id == CODEC_ID_XAN_WC3) &&
(s->avctx->palctrl == NULL)) { (s->avctx->palctrl == NULL)) {
av_log(avctx, AV_LOG_ERROR, " WC3 Xan video: palette expected.\n"); av_log(avctx, AV_LOG_ERROR, " WC3 Xan video: palette expected.\n");
return -1; return AVERROR(EINVAL);
} }
avctx->pix_fmt = PIX_FMT_PAL8; 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_size = avctx->width * avctx->height;
s->buffer1 = av_malloc(s->buffer1_size); s->buffer1 = av_malloc(s->buffer1_size);
if (!s->buffer1) if (!s->buffer1)
return -1; return AVERROR(ENOMEM);
s->buffer2_size = avctx->width * avctx->height; s->buffer2_size = avctx->width * avctx->height;
s->buffer2 = av_malloc(s->buffer2_size + 130); s->buffer2 = av_malloc(s->buffer2_size + 130);
if (!s->buffer2) { if (!s->buffer2) {
av_freep(&s->buffer1); av_freep(&s->buffer1);
return -1; return AVERROR(ENOMEM);
} }
return 0; return 0;
@ -359,13 +359,13 @@ static int xan_decode_frame(AVCodecContext *avctx,
AVPacket *avpkt) AVPacket *avpkt)
{ {
const uint8_t *buf = avpkt->data; const uint8_t *buf = avpkt->data;
int buf_size = avpkt->size; int ret, buf_size = avpkt->size;
XanContext *s = avctx->priv_data; XanContext *s = avctx->priv_data;
AVPaletteControl *palette_control = avctx->palctrl; 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"); av_log(s->avctx, AV_LOG_ERROR, " Xan Video: get_buffer() failed\n");
return -1; return ret;
} }
s->current_frame.reference = 3; s->current_frame.reference = 3;

Loading…
Cancel
Save