On Tue, Sep 02, 2014 at 09:54:31PM +0200, Gabriel Dume wrote: > ffmpeg | branch: master | Gabriel Dume <[email protected]> | Mon Sep 1 > 15:18:57 2014 -0400| [74512f7e369da40e1148c92b58cd8e59f7737b8f] | committer: > Diego Biurrun > > 8svx: Return proper error codes > > Signed-off-by: Diego Biurrun <[email protected]> > > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=74512f7e369da40e1148c92b58cd8e59f7737b8f > --- > > libavcodec/8svx.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/libavcodec/8svx.c b/libavcodec/8svx.c > index 11fbf19..b973771 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;
Isn't this a user settable option? EINVAL was appropriate then.
> }
>
> 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;
AVERROR_BUG, or EINVAL maybe?
Unless I'm mistaken this is set depending on the selected decoder. It
looks unrelated to the stream data anyway.
[...]
--
Clément B.
pgpjQTmptg957.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
