diff options
author | James Almer <jamrial@gmail.com> | 2018-02-26 00:18:53 -0300 |
---|---|---|
committer | James Almer <jamrial@gmail.com> | 2018-02-26 22:17:21 -0300 |
commit | f235359b2b1175585df757e6b60fabac29b8aa22 (patch) | |
tree | 0e35af0b8bd16422333442c3d8939d677efff64e /libavformat | |
parent | f4709f1b7b9008380a74dd0415323fd3e0085c93 (diff) | |
download | ffmpeg-f235359b2b1175585df757e6b60fabac29b8aa22.tar.gz |
avformat/utils: don't overwrite the return value of read_packet()
This only affected demuxers that didn't return reference counted packets.
Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: James Almer <jamrial@gmail.com>
Diffstat (limited to 'libavformat')
-rw-r--r-- | libavformat/utils.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libavformat/utils.c b/libavformat/utils.c index 2c2ea876b6..72531d4185 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -869,9 +869,9 @@ int ff_read_packet(AVFormatContext *s, AVPacket *pkt) if (!pkt->buf) { AVPacket tmp = { 0 }; - ret = av_packet_ref(&tmp, pkt); - if (ret < 0) - return ret; + err = av_packet_ref(&tmp, pkt); + if (err < 0) + return err; *pkt = tmp; } |