Hi Diego,
On Mon, Jul 17, 2017 at 4:46 AM, Diego Biurrun <di...@biurrun.de> wrote:
> On Sun, Jul 16, 2017 at 12:24:06PM -0400, Sean McGovern wrote:
>> --- a/libavformat/rmdec.c
>> +++ b/libavformat/rmdec.c
>> @@ -778,11 +779,17 @@ ff_rm_parse_packet (AVFormatContext *s, AVIOContext
>> *pb,
>> } else {
>> - av_get_packet(pb, pkt, len);
>> + ret = av_get_packet(pb, pkt, len);
>> + if (ret < 0) {
>> + return ret;
>> + }
>> rm_ac3_swap_bytes(st, pkt);
>> }
>> } else
>> - av_get_packet(pb, pkt, len);
>> + ret = av_get_packet(pb, pkt, len);
>> + if (ret < 0) {
>> + return ret;
>> + }
>>
>> @@ -798,14 +805,19 @@ ff_rm_retrieve_cache (AVFormatContext *s, AVIOContext
>> *pb,
>> if (ast->deint_id == DEINT_ID_VBRF ||
>> - ast->deint_id == DEINT_ID_VBRS)
>> - av_get_packet(pb, pkt, ast->sub_packet_lengths[ast->sub_packet_cnt
>> - rm->audio_pkt_cnt]);
>> + ast->deint_id == DEINT_ID_VBRS) {
>> + ret = av_get_packet(pb, pkt,
>> ast->sub_packet_lengths[ast->sub_packet_cnt - rm->audio_pkt_cnt]);
>> + if (ret < 0) {
>> + return ret;
>> + }
>
> Drop the unnecessary {} ..
>
Amended locally and will push shortly.
-- Sean McG.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
