aboutsummaryrefslogtreecommitdiffstats
path: root/libavformat/rmdec.c
diff options
context:
space:
mode:
authorJames Almer <jamrial@gmail.com>2017-11-07 19:15:16 -0300
committerJames Almer <jamrial@gmail.com>2017-11-07 19:17:26 -0300
commitfe421d4deecd12e498e6f07b3fb6f83212c007b2 (patch)
treeed37fdaa2f87cdfeacb2a8163515103057137d94 /libavformat/rmdec.c
parent85b9fe9584145bde72dbe2e24d2eee513f198256 (diff)
parent95ce02b35d3d1bb16111031df1d82a6e4d894d36 (diff)
downloadffmpeg-fe421d4deecd12e498e6f07b3fb6f83212c007b2.tar.gz
Merge commit '95ce02b35d3d1bb16111031df1d82a6e4d894d36'
* commit '95ce02b35d3d1bb16111031df1d82a6e4d894d36': rmdec: don't ignore the return value of av_get_packet() See 65b83ce01b94eae944290c11257799a9288f2d4c Merged-by: James Almer <jamrial@gmail.com>
Diffstat (limited to 'libavformat/rmdec.c')
-rw-r--r--libavformat/rmdec.c11
1 files changed, 7 insertions, 4 deletions
diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c
index d6d7d9cd84..fea71a2fc9 100644
--- a/libavformat/rmdec.c
+++ b/libavformat/rmdec.c
@@ -950,12 +950,14 @@ ff_rm_parse_packet (AVFormatContext *s, AVIOContext *pb,
} else
return -1;
} else {
- if ((ret = av_get_packet(pb, pkt, len)) < 0)
+ ret = av_get_packet(pb, pkt, len);
+ if (ret < 0)
return ret;
rm_ac3_swap_bytes(st, pkt);
}
} else {
- if ((ret = av_get_packet(pb, pkt, len)) < 0)
+ ret = av_get_packet(pb, pkt, len);
+ if (ret < 0)
return ret;
}
@@ -973,16 +975,17 @@ ff_rm_retrieve_cache (AVFormatContext *s, AVIOContext *pb,
AVStream *st, RMStream *ast, AVPacket *pkt)
{
RMDemuxContext *rm = s->priv_data;
+ int ret;
av_assert0 (rm->audio_pkt_cnt > 0);
if (ast->deint_id == DEINT_ID_VBRF ||
ast->deint_id == DEINT_ID_VBRS) {
- int ret = av_get_packet(pb, pkt, ast->sub_packet_lengths[ast->sub_packet_cnt - rm->audio_pkt_cnt]);
+ ret = av_get_packet(pb, pkt, ast->sub_packet_lengths[ast->sub_packet_cnt - rm->audio_pkt_cnt]);
if (ret < 0)
return ret;
} else {
- int ret = av_new_packet(pkt, st->codecpar->block_align);
+ ret = av_new_packet(pkt, st->codecpar->block_align);
if (ret < 0)
return ret;
memcpy(pkt->data, ast->pkt.data + st->codecpar->block_align * //FIXME avoid this