[FFmpeg-cvslog] avcodec/mmaldec: re-use AVPacket for extra_data

Ho Ming Shun git at videolan.org
Sun Dec 12 18:02:55 EET 2021


ffmpeg | branch: master | Ho Ming Shun <cyph1984 at gmail.com> | Fri Sep 24 17:04:37 2021 +0800| [6f0e8b998a91f0ad9a260b01d8ff29efdf3321d3] | committer: Andreas Rheinhardt

avcodec/mmaldec: re-use AVPacket for extra_data

extra_data and normal packets (from ff_decode_get_packet) processing do
not overlap, thus we can re-use the spare AVPacket to send to
ffmmal_add_packet.

Furthermore, this removes allocation of AVPacket on the stack and stops
using deprecated av_init_packet.

Tested-by: Cameron Gutman <aicommander at gmail.com>
Signed-off-by: Ho Ming Shun <cyph1984 at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=6f0e8b998a91f0ad9a260b01d8ff29efdf3321d3
---

 libavcodec/mmaldec.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/libavcodec/mmaldec.c b/libavcodec/mmaldec.c
index f0391dfae3..f1bcaed0ff 100644
--- a/libavcodec/mmaldec.c
+++ b/libavcodec/mmaldec.c
@@ -782,12 +782,10 @@ static int ffmmal_receive_frame(AVCodecContext *avctx, AVFrame *frame)
     int got_frame = 0;
 
     if (avctx->extradata_size && !ctx->extradata_sent) {
-        AVPacket pkt = {0};
-        av_init_packet(&pkt);
-        pkt.data = avctx->extradata;
-        pkt.size = avctx->extradata_size;
+        avpkt->data = avctx->extradata;
+        avpkt->size = avctx->extradata_size;
         ctx->extradata_sent = 1;
-        if ((ret = ffmmal_add_packet(avctx, &pkt, 1)) < 0)
+        if ((ret = ffmmal_add_packet(avctx, avpkt, 1)) < 0)
             return ret;
     }
 



More information about the ffmpeg-cvslog mailing list