diff options
author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-01-07 16:55:40 +0300 |
---|---|---|
committer | Marton Balint <cus@passwd.hu> | 2020-02-11 00:41:38 +0300 |
commit | 6a67d518d60b61ae2bd98c3948894a7be9955e8c (patch) | |
tree | 25f165390e6e0e72537007ee31edc7a03a02b789 /libavformat/dfa.c | |
parent | bbea268aa806a740e25c7dededf8dbe946e78bc5 (diff) |
avformat: Remove unnecessary av_packet_unref()
Since bae8844e the packet will always be unreferenced when a demuxer
returns an error, so that a lot of calls to av_packet_unref() in lots of
demuxers are now redundant and can be removed.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Signed-off-by: Marton Balint <cus@passwd.hu>
Diffstat (limited to 'libavformat/dfa.c')
-rw-r--r-- | libavformat/dfa.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/libavformat/dfa.c b/libavformat/dfa.c index d667cd6277..bd4ef2dafe 100644 --- a/libavformat/dfa.c +++ b/libavformat/dfa.c @@ -93,7 +93,6 @@ static int dfa_read_packet(AVFormatContext *s, AVPacket *pkt) if (!first) { ret = av_append_packet(pb, pkt, 12); if (ret < 0) { - av_packet_unref(pkt); return ret; } } else @@ -101,7 +100,6 @@ static int dfa_read_packet(AVFormatContext *s, AVPacket *pkt) frame_size = AV_RL32(pkt->data + pkt->size - 8); if (frame_size > INT_MAX - 4) { av_log(s, AV_LOG_ERROR, "Too large chunk size: %"PRIu32"\n", frame_size); - av_packet_unref(pkt); return AVERROR(EIO); } if (AV_RL32(pkt->data + pkt->size - 12) == MKTAG('E', 'O', 'F', 'R')) { @@ -115,7 +113,6 @@ static int dfa_read_packet(AVFormatContext *s, AVPacket *pkt) } ret = av_append_packet(pb, pkt, frame_size); if (ret < 0) { - av_packet_unref(pkt); return ret; } } |