[FFmpeg-devel] [PATCH] avcodec/alsdec: treat quant_cof as a signed value

Michael Niedermayer michael at niedermayer.cc
Sun Mar 6 13:04:29 CET 2016


On Sun, Mar 06, 2016 at 09:00:47AM +0100, Paul B Mahol wrote:
> On 3/5/16, Umair Khan <omerjerk at gmail.com> wrote:
> > Signed-off-by: Umair Khan <omerjerk at gmail.com>
> > ---
> >  libavcodec/alsdec.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c
> > index ebd364e..0043512 100644
> > --- a/libavcodec/alsdec.c
> > +++ b/libavcodec/alsdec.c
> > @@ -729,7 +729,7 @@ static int read_var_block_data(ALSDecContext *ctx,
> > ALSBlockData *bd)
> >                      quant_cof[k] = decode_rice(gb, rice_param) + offset;
> >                      if (quant_cof[k] < -64 || quant_cof[k] > 63) {
> >                          av_log(avctx, AV_LOG_ERROR,
> > -                               "quant_cof %"PRIu32" is out of range.\n",
> > +                               "quant_cof %"PRId32" is out of range.\n",
> >                                 quant_cof[k]);
> >                          return AVERROR_INVALIDDATA;
> >                      }
> > --
> > 2.5.0
> 
> LGTM

applied

thanks

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

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160306/c12bf6b9/attachment.sig>


More information about the ffmpeg-devel mailing list