summaryrefslogtreecommitdiff
path: root/libavformat/dss.c
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-01-07 14:55:40 +0100
committerMarton Balint <cus@passwd.hu>2020-02-10 22:41:38 +0100
commit6a67d518d60b61ae2bd98c3948894a7be9955e8c (patch)
tree25f165390e6e0e72537007ee31edc7a03a02b789 /libavformat/dss.c
parentbbea268aa806a740e25c7dededf8dbe946e78bc5 (diff)
downloadffmpeg-6a67d518d60b61ae2bd98c3948894a7be9955e8c.tar.gz
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/dss.c')
-rw-r--r--libavformat/dss.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/libavformat/dss.c b/libavformat/dss.c
index d7f9cafe47..8e0b72fbe5 100644
--- a/libavformat/dss.c
+++ b/libavformat/dss.c
@@ -259,14 +259,12 @@ static int dss_sp_read_packet(AVFormatContext *s, AVPacket *pkt)
dss_sp_byte_swap(ctx, pkt->data, ctx->dss_sp_buf);
if (ctx->dss_sp_swap_byte < 0) {
- ret = AVERROR(EAGAIN);
- goto error_eof;
+ return AVERROR(EAGAIN);
}
return pkt->size;
error_eof:
- av_packet_unref(pkt);
return ret < 0 ? ret : AVERROR_EOF;
}
@@ -308,7 +306,6 @@ static int dss_723_1_read_packet(AVFormatContext *s, AVPacket *pkt)
ret = avio_read(s->pb, pkt->data + offset,
size2 - offset);
if (ret < size2 - offset) {
- av_packet_unref(pkt);
return ret < 0 ? ret : AVERROR_EOF;
}
@@ -318,7 +315,6 @@ static int dss_723_1_read_packet(AVFormatContext *s, AVPacket *pkt)
ret = avio_read(s->pb, pkt->data + offset, size - offset);
if (ret < size - offset) {
- av_packet_unref(pkt);
return ret < 0 ? ret : AVERROR_EOF;
}