On 2/5/15, Michael Niedermayer <michae...@gmx.at> wrote:
> On Thu, Feb 05, 2015 at 03:08:17PM +0000, Paul B Mahol wrote:
>> Signed-off-by: Paul B Mahol <one...@gmail.com>
>> ---
>> libavformat/rpl.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/libavformat/rpl.c b/libavformat/rpl.c
>> index c1229e8..a05bff1 100644
>> --- a/libavformat/rpl.c
>> +++ b/libavformat/rpl.c
>> @@ -308,6 +308,8 @@ static int rpl_read_packet(AVFormatContext *s,
>> AVPacket *pkt)
>> return AVERROR(EIO);
>>
>> ret = av_get_packet(pb, pkt, frame_size);
>> + if (ret < 0)
>> + return ret;
>> if (ret != frame_size) {
>> av_free_packet(pkt);
>> return AVERROR(EIO);
>> @@ -323,6 +325,8 @@ static int rpl_read_packet(AVFormatContext *s,
>> AVPacket *pkt)
>> }
>> } else {
>> ret = av_get_packet(pb, pkt, index_entry->size);
>> + if (ret < 0)
>> + return ret;
>
> ret is unsigned, so this cannot be true
Should I then make ret signed?
>
> [...]
> --
> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> The real ebay dictionary, page 1
> "Used only once" - "Some unspecified defect prevented a second use"
> "In good condition" - "Can be repaird by experienced expert"
> "As is" - "You wouldnt want it even if you were payed for it, if you knew
> ..."
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
