From: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Date: Sun, 30 Aug 2020 05:32:48 +0000 (+0200)
Subject: avdevice/lavfi: Don't set pkt->size to the value it already has
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=901251d59bd287954f334bc153a0d71853debb92;p=ffmpeg

avdevice/lavfi: Don't set pkt->size to the value it already has

av_new_packet() already sets the size. And if the packet is not
allocated by av_new_packet() (which seems to be impossible atm), both
pkt->size as well as size are 0, so setting it again is unnecessary in
this scenario, too.

Reviewed-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---

diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
index 5e814eada85..72ec1bb9b7c 100644
--- a/libavdevice/lavfi.c
+++ b/libavdevice/lavfi.c
@@ -474,7 +474,6 @@ static int lavfi_read_packet(AVFormatContext *avctx, AVPacket *pkt)
     pkt->stream_index = stream_idx;
     pkt->pts = frame->pts;
     pkt->pos = frame->pkt_pos;
-    pkt->size = size;
     av_frame_unref(frame);
     return size;
 }