2012/1/6 Justin Ruggles <[email protected]>

> On 01/06/2012 01:42 PM, Laurentiu Ion wrote:
>
> > Prevent invalid reads using bytestream2 functions
> > ---
> >  libavcodec/flicvideo.c |  161
> +++++++++++++++++++++++-------------------------
> >  1 files changed, 76 insertions(+), 85 deletions(-)
> >
> > diff --git a/libavcodec/flicvideo.c b/libavcodec/flicvideo.c
> > index 8e8a813..d96375a 100644
> > --- a/libavcodec/flicvideo.c
> > +++ b/libavcodec/flicvideo.c
>
>
> there are a few cosmetic nit-pick issues, but overall i think the patch
> looks fine. and i assume it passes the 3 flic fate tests and that you
> tested that it indeed fixes bug 126?
>
> -Justin
> _______________________________________________
> libav-devel mailing list
> [email protected]
> https://lists.libav.org/mailman/listinfo/libav-devel


 yes, tested on all 3 FATE flic tests, and valgrind output for the fuzzed
file is fine: http://pastebin.com/ySeKSzJ7
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to