On Thu, Sep 3, 2015 at 12:12 PM, Alexandra Hájková
<[email protected]> wrote:
> On Thu, Sep 3, 2015 at 12:09 PM, Luca Barbato <[email protected]> wrote:
>> On 03/09/15 10:08, Alexandra Hájková wrote:
>>> @@ -1141,7 +1141,7 @@ static int asf_read_replicated_data(AVFormatContext 
>>> *s, ASFPacket *asf_pkt)
>>>      if (!asf_pkt->data_size) {
>>>          asf_pkt->data_size = asf_pkt->size_left = avio_rl32(pb); // read 
>>> media object size
>>>          if (asf_pkt->data_size <= 0)
>>> -            return AVERROR_EOF;
>>> +            return AVERROR_INVALIDDATA;
>>
>> Looks a stray hunk. The rest seems ok.
>
> I wanted all the checks to be consistent

nothing wrong with that, it should just be mentioned in the commit log
i think a line could be added by whoever pushes this.
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to