[libav-commits] avcodec/extract_extradata: Move the reference in the bsf internal buffer

[James Almer]

Module: libav
Branch: master
Commit: 6442d8bad68052a1d42565fbbad85695fadd26c1

Author:    James Almer <jamr...@gmail.com>
Committer: Luca Barbato <lu_z...@gentoo.org>
Date:      Sun Mar 11 13:45:05 2018 -0300

avcodec/extract_extradata: Move the reference in the bsf internal buffer

There is no need to allocate a new packet for it.

Reviewed-by: Mark Thompson <s...@jkqxz.net>
Signed-off-by: James Almer <jamr...@gmail.com>
Signed-off-by: Luca Barbato <lu_z...@gentoo.org>

---

 libavcodec/extract_extradata_bsf.c | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/libavcodec/extract_extradata_bsf.c 
b/libavcodec/extract_extradata_bsf.c
index 90ff8c8..d2773ce 100644
--- a/libavcodec/extract_extradata_bsf.c
+++ b/libavcodec/extract_extradata_bsf.c
@@ -252,24 +252,23 @@ static int extract_extradata_init(AVBSFContext *ctx)
     return 0;
 }
 
-static int extract_extradata_filter(AVBSFContext *ctx, AVPacket *out)
+static int extract_extradata_filter(AVBSFContext *ctx, AVPacket *pkt)
 {
     ExtractExtradataContext *s = ctx->priv_data;
-    AVPacket *in;
     uint8_t *extradata = NULL;
     int extradata_size;
     int ret = 0;
 
-    ret = ff_bsf_get_packet(ctx, &in);
+    ret = ff_bsf_get_packet_ref(ctx, pkt);
     if (ret < 0)
         return ret;
 
-    ret = s->extract(ctx, in, &extradata, &extradata_size);
+    ret = s->extract(ctx, pkt, &extradata, &extradata_size);
     if (ret < 0)
         goto fail;
 
     if (extradata) {
-        ret = av_packet_add_side_data(in, AV_PKT_DATA_NEW_EXTRADATA,
+        ret = av_packet_add_side_data(pkt, AV_PKT_DATA_NEW_EXTRADATA,
                                       extradata, extradata_size);
         if (ret < 0) {
             av_freep(&extradata);
@@ -277,10 +276,10 @@ static int extract_extradata_filter(AVBSFContext *ctx, 
AVPacket *out)
         }
     }
 
-    av_packet_move_ref(out, in);
+    return 0;
 
 fail:
-    av_packet_free(&in);
+    av_packet_unref(pkt);
     return ret;
 }
 

_______________________________________________
libav-commits mailing list
libav-commits@libav.org
https://lists.libav.org/mailman/listinfo/libav-commits