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.
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to