]> git.sesse.net Git - ffmpeg/commitdiff
avformat/adp, svs: Remove redundant av_shrink_packet()
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Fri, 12 Mar 2021 08:23:46 +0000 (09:23 +0100)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Fri, 12 Mar 2021 16:13:10 +0000 (17:13 +0100)
av_get_packet() already makes sure that the packet size is accurate
and that the packet data is zero-padded even when one could not read as
much as desired.

Reviewed-by: James Almer <jamrial@gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavformat/adp.c
libavformat/svs.c

index 8668c78fe43c8b835b6b1c27a2733ed00eb99d2a..b0ee09753e48165512600d3a504038fc79c0ab5b 100644 (file)
@@ -75,13 +75,9 @@ static int adp_read_packet(AVFormatContext *s, AVPacket *pkt)
         return AVERROR_EOF;
 
     ret = av_get_packet(s->pb, pkt, size);
+    if (ret < 0)
+        return ret;
 
-    if (ret != size) {
-        if (ret < 0) {
-            return ret;
-        }
-        av_shrink_packet(pkt, ret);
-    }
     pkt->stream_index = 0;
 
     return ret;
index d4285ed3069b8ba569bb187635e0d77cf1b8b6da..8be26c5bc33401a1a47af7286436412d0cabf1ac 100644 (file)
@@ -79,7 +79,6 @@ static int svs_read_packet(AVFormatContext *s, AVPacket *pkt)
     if (ret != 32 * 256) {
         if (ret < 0)
             return ret;
-        av_shrink_packet(pkt, ret);
         pkt->flags &= ~AV_PKT_FLAG_CORRUPT;
     }
     pkt->stream_index = 0;