[FFmpeg-devel] [PATCH] Make g729dec.c:decoder_init() return AVERROR(EINVAL) rather than AVERROR(NOFMT) in case of ac != 1

Michael Niedermayer michaelni
Wed Mar 17 15:49:05 CET 2010


On Wed, Mar 17, 2010 at 12:55:47AM +0100, Stefano Sabatini wrote:
> $subj.
> 
> This set the interesting problem if an invalid value set in the
> context should be interpreted as:
> 
> EINVAL: Invalid argument
> EDOM: Numerical argument out of domain
> INVALID_DATA
> EPERM: Operation not permitted (aka -1, this is what it looks is done in most of the code)
> 
> something else?
> 
> Regards.
> -- 
> FFmpeg = Fabulous Funny Mega Philosophical Emblematic Gigant

>  g729dec.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 9dbc7e0481068f47dd2446ec43b473ed98e89dae  0008-Make-decoder_init-return-AVERROR-EINVAL-rather-than-.patch
> >From 707ab5ff5f3255dacd4bf7a201ec851debeb274e Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> Date: Wed, 17 Mar 2010 00:14:46 +0100
> Subject: [PATCH 08/15] Make decoder_init() return AVERROR(EINVAL) rather than AVERROR(NOFMT)
>  if one channel if the number of channels specified is different than
>  1.

should be ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100317/dc985cc8/attachment.pgp>



More information about the ffmpeg-devel mailing list