On Fri, Oct 14, 2011 at 02:30:49PM -0700, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 14, 2011 at 2:16 PM, Janne Grunau <[email protected]> wrote: > > On Fri, Oct 14, 2011 at 11:57:28AM -0700, Ronald S. Bultje wrote: > >> See attached. > >> > >> Ronald > > > >> Index: orig/libavformat/matroskadec.c > >> =================================================================== > >> --- orig/libavformat/matroskadec.c (revision 105133) > >> +++ ffmpeg/libavformat/matroskadec.c (working copy) > >> @@ -1841,7 +1841,7 @@ > >> lace_size[n] = lace_size[n - 1] + snum; > >> total += lace_size[n]; > >> } > >> - lace_size[n] = size - total; > >> + lace_size[laces - 1] = size - total; > > > > if I found the right place this patch should apply to this change doesn't > > make a difference, since in our master branch n is already laces-1 > > Not true, since n is initialized to 1 and if laces is 1, laces-1 is > zero but n is still 1.
yes, haven't thought of laces being 1, patch ok and queued Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
