[FFmpeg-devel] [PATCH 1/6] avcodec/aacdec_template: Check samplerate

Michael Niedermayer michael at niedermayer.cc
Thu Oct 10 15:20:04 EEST 2019


On Thu, Oct 10, 2019 at 12:06:43PM +0200, Moritz Barsnick wrote:
> On Wed, Oct 09, 2019 at 12:14:49 +0200, Michael Niedermayer wrote:
> > +    if (avctx->sample_rate > INT_MAX/2)
> > +        return AVERROR_INVALIDDATA;
> 
> Who enforces the defined sample rates? The demuxer?
> 
> I'm trying to say: The maxmimum for AAC is 96000, why not use that?
> (Will it ever change? Can it?)

It certainly could, limits can always change in a revission or extension
but till that happens 96000 is a good choice as limit so ill change it
to that

Thanks


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

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20191010/f104155e/attachment.sig>


More information about the ffmpeg-devel mailing list