[FFmpeg-devel] [PATCH]Simplify als error condition

Carl Eugen Hoyos cehoyos at ag.or.at
Sat May 5 13:26:44 CEST 2012


Thilo Borgmann <thilo.borgmann <at> googlemail.com> writes:

> >      for (k = 1; k < sub_blocks; k++)
> 
> I think this loop should also check s[k] for k == 0. Would be nice 
> if you could also push that by another commit

Done.
(Hope it's correct.)

> > -        if (s[k] < 0 || s[k] > 32) {
> > +        if (s[k] > 32) {
> >              av_log(avctx, AV_LOG_ERROR, "k invalid for rice code.\n");
> >              return -1;
> >          }
> 
> patch ok.

Patch applied.

Thank you, Carl Eugen



More information about the ffmpeg-devel mailing list