On Fri, Oct 21, 2011 at 01:41:36PM +0300, Jan E wrote: > From: Michael Niedermayer <[email protected]> > > This inconsistency was noticed during debug. > I dont know if this can happen normally, but if sit can, > it would be quite bad. > > Signed-off-by: Michael Niedermayer <[email protected]> > --- > libavcodec/aacsbr.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c > index 81b0b4c..2910395 100644 > --- a/libavcodec/aacsbr.c > +++ b/libavcodec/aacsbr.c > @@ -33,6 +33,7 @@ > #include "fft.h" > #include "aacps.h" > #include "libavutil/libm.h" > +#include "libavutil/avassert.h" > > #include <stdint.h> > #include <float.h> > @@ -1456,6 +1457,7 @@ static void sbr_mapping(AACContext *ac, > SpectralBandReplication *sbr, > uint16_t *table = ch_data->bs_freq_res[e + 1] ? sbr->f_tablehigh : > sbr->f_tablelow; > int k; > > + av_assert0(sbr->kx[1] <= table[0]); > for (i = 0; i < ilim; i++) > for (m = table[i]; m < table[i + 1]; m++) > sbr->e_origmapped[e][m - sbr->kx[1]] = > ch_data->env_facs[e+1][i];
a bit pointless to check just one value without knowing more under which conditions it occured. typos in the commit message should fixed before pushing. Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
