well, before I was told the commit message was too long... this fixes the
flicvideo.c invalid reads, using the bytestream2 functions... I tested with
the FATE tests for flic and all passed, all I need is for you guys to
review it

2012/1/6 Reinhard Tartler <[email protected]>

> On Fr, Jan 06, 2012 at 16:54:40 (CET), Laurentiu Ion wrote:
>
> > ---
> >  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
> > @@ -41,6 +41,8 @@
>
> [...]
>
> poor commit message. please try to be more descriptive what this change
> does and how a potential reviewer is supposed to verify it.
>
> Cheers,
> Reinhard
>
> --
> Gruesse/greetings,
> Reinhard Tartler, KeyID 945348A4
> _______________________________________________
> libav-devel mailing list
> [email protected]
> https://lists.libav.org/mailman/listinfo/libav-devel
>
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to