diff options
| author | Gabriel Dume <gabriel.ddx84@gmail.com> | 2014-09-01 15:18:57 -0400 |
|---|---|---|
| committer | Diego Biurrun <diego@biurrun.de> | 2014-09-02 03:42:13 -0700 |
| commit | 74512f7e369da40e1148c92b58cd8e59f7737b8f (patch) | |
| tree | 980f924e017ad92eecfbeaedcf79cbdc9443457f /libavcodec/8svx.c | |
| parent | e48d1ea541be4592ebac89875557407ca958b7a9 (diff) | |
8svx: Return proper error codes
Signed-off-by: Diego Biurrun <diego@biurrun.de>
Diffstat (limited to 'libavcodec/8svx.c')
| -rw-r--r-- | libavcodec/8svx.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/libavcodec/8svx.c b/libavcodec/8svx.c index 11fbf193ce..b973771fa5 100644 --- a/libavcodec/8svx.c +++ b/libavcodec/8svx.c @@ -96,11 +96,11 @@ static int eightsvx_decode_frame(AVCodecContext *avctx, void *data, if (avpkt->size < hdr_size * avctx->channels) { av_log(avctx, AV_LOG_ERROR, "packet size is too small\n"); - return AVERROR(EINVAL); + return AVERROR_INVALIDDATA; } if (esc->data[0]) { av_log(avctx, AV_LOG_ERROR, "unexpected data after first packet\n"); - return AVERROR(EINVAL); + return AVERROR_INVALIDDATA; } if (is_compr) { @@ -125,7 +125,7 @@ static int eightsvx_decode_frame(AVCodecContext *avctx, void *data, } if (!esc->data[0]) { av_log(avctx, AV_LOG_ERROR, "unexpected empty packet\n"); - return AVERROR(EINVAL); + return AVERROR_INVALIDDATA; } /* decode next piece of data from the buffer */ @@ -166,7 +166,7 @@ static av_cold int eightsvx_decode_init(AVCodecContext *avctx) if (avctx->channels < 1 || avctx->channels > 2) { av_log(avctx, AV_LOG_ERROR, "8SVX does not support more than 2 channels\n"); - return AVERROR(EINVAL); + return AVERROR_INVALIDDATA; } switch(avctx->codec->id) { @@ -179,7 +179,7 @@ static av_cold int eightsvx_decode_init(AVCodecContext *avctx) case AV_CODEC_ID_PCM_S8_PLANAR: break; default: - return -1; + return AVERROR_INVALIDDATA; } avctx->sample_fmt = AV_SAMPLE_FMT_U8P; |