On 21/10/14 15:15, Vittorio Giovara wrote:
> On Mon, Oct 20, 2014 at 10:32 PM, Luca Barbato <[email protected]> wrote:
>> On 20/10/14 15:11, Vittorio Giovara wrote:
>>> CC: [email protected]
>>> Bug-Id: CID 90558
>>> ---
>>>  libavformat/rmdec.c | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c
>>> index 79a0756..97f164f 100644
>>> --- a/libavformat/rmdec.c
>>> +++ b/libavformat/rmdec.c
>>> @@ -863,7 +863,7 @@ static int rm_read_packet(AVFormatContext *s, AVPacket 
>>> *pkt)
>>>                      st = s->streams[i];
>>>              }
>>>
>>> -            if(len<0 || s->pb->eof_reached)
>>> +            if (len <= 0 || s->pb->eof_reached)
>>>                  return AVERROR(EIO);
>>>
>>>              res = ff_rm_parse_packet (s, s->pb, st, st->priv_data, len, 
>>> pkt,
>>>
>>
>> Probably ok, assuming real doesn't allow 0-sized packets.
> 
> Even if it did, I am quite sure that av_get_packet and
> rm_assemble_video_frame would complain about it.
> 
Probably Ok then, Katerina might tell us more about it =)

lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to