[FFmpeg-devel] [PATCH] avformat/riffdec: remove invalid sample rate check

[Michael Niedermayer]

On Sat, Jan 25, 2025 at 02:19:52AM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Tue, Jan 21, 2025 at 05:30:05PM +0530, Viraaj Raulgaonkar wrote:
> > Sample rates up to 2^31 - 1 are considered valid, while greater values
> > are invalid.  Removing the check potentially allows the sample rate
> > to be decoded in all cases instead of halting demux for values
> > greater than 2^31 - 1.
> > 
> > Fixes Trac ticket #11361.
> > ---
> >  libavformat/riffdec.c | 5 -----
> >  1 file changed, 5 deletions(-)
> > 
> > diff --git a/libavformat/riffdec.c b/libavformat/riffdec.c
> > index b7a85a6ab2..94dccb324e 100644
> > --- a/libavformat/riffdec.c
> > +++ b/libavformat/riffdec.c
> > @@ -179,11 +179,6 @@ int ff_get_wav_header(void *logctx, AVIOContext *pb,
> >  
> >      par->bit_rate = bitrate;
> >  
> > -    if (par->sample_rate <= 0) {
> > -        av_log(logctx, AV_LOG_ERROR,
> > -               "Invalid sample rate: %d\n", par->sample_rate);
> > -        return AVERROR_INVALIDDATA;
> > -    }
> 
> the API does not really allow negative sample rates

If you want to export the stored sample rate value correctly, this would
need a API change (which is a alot of work and seems not justified here)

Otherwise, simply print a warning and return sample_rate/2 instead
(assuming that works, does it work ?)

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Some people wanted to paint the bikeshed green, some blue and some pink.
People argued and fought, when they finally agreed, only rust was left.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250125/ebc3ae4c/attachment.sig>