oh and the bug report is http://bugzilla.libav.org/attachment.cgi?bugid=126

2012/1/6 Laurentiu Cristian Ion <[email protected]>

> 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