[FFmpeg-devel] [PATCH] avformat/riffdec: remove invalid sample rate check
Michael Niedermayer
michael at niedermayer.cc
Sat Jan 25 03:26:27 EET 2025
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>
More information about the ffmpeg-devel
mailing list