On Fri, Mar 23, 2012 at 12:34:47PM -0700, Alex Converse wrote: > On Fri, Mar 23, 2012 at 12:23 PM, Vitor Sessak <[email protected]> wrote: > > On 03/23/2012 07:17 AM, Kostya Shishkov wrote: > >> > >> On Thu, Mar 22, 2012 at 01:46:13PM -0700, Ronald S. Bultje wrote: > >>> > >>> Hi, > >>> > >>> On Thu, Mar 22, 2012 at 1:08 PM, Kostya Shishkov > >>> <[email protected]> wrote: > >>>> > >>>> On Thu, Mar 22, 2012 at 12:55:38PM -0700, Ronald S. Bultje wrote: > >>>>> > >>>>> From: "Ronald S. Bultje"<[email protected]> > >>>>> > >>>>> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind > >>>>> --- > >>>>> libavcodec/ralfdata.h | 12 ++++++------ > >>>>> 1 file changed, 6 insertions(+), 6 deletions(-) > >>>> > >>>> > >>>> It's senseless IMO - especially with current bitstream reader. > >>> > >>> > >>> The current bitstream reader does overread, but by a guaranteed amount > >>> of less than FF_INPUT_BUFFER_PADDING_BYTES bytes. > >> > >> > >> OK, but don't forget a proper #include for FF_INPUT_BUFFER_PADDING_BYTES, > >> or it will fail make checkheaders and make Diego unhappier. > > > > > > Wouldn't this make the huge tables even bigger? Another solution would be > > not to use the bitstream reader... > > > > It looks like these tables are read one half byte at a time so I agree > with Vitor.
I can look at it tomorrow if Ronald doesn't beat me to it. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
