[FFmpeg-cvslog] avcodec/bsf: use av_packet_make_refcounted to ensure packets are ref counted

James Almer git at videolan.org
Mon Apr 2 06:50:08 EEST 2018


ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Sun Mar 25 00:23:58 2018 -0300| [265ec55983bf3b981190b564925a609d53391aa5] | committer: James Almer

avcodec/bsf: use av_packet_make_refcounted to ensure packets are ref counted

Simplifies code.

Signed-off-by: James Almer <jamrial at gmail.com>

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

 libavcodec/bsf.c | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/libavcodec/bsf.c b/libavcodec/bsf.c
index 05a44e2e31..bd611ea16b 100644
--- a/libavcodec/bsf.c
+++ b/libavcodec/bsf.c
@@ -174,6 +174,8 @@ int av_bsf_init(AVBSFContext *ctx)
 
 int av_bsf_send_packet(AVBSFContext *ctx, AVPacket *pkt)
 {
+    int ret;
+
     if (!pkt || (!pkt->data && !pkt->side_data_elems)) {
         ctx->internal->eof = 1;
         return 0;
@@ -188,15 +190,10 @@ int av_bsf_send_packet(AVBSFContext *ctx, AVPacket *pkt)
         ctx->internal->buffer_pkt->side_data_elems)
         return AVERROR(EAGAIN);
 
-    if (pkt->buf) {
-        av_packet_move_ref(ctx->internal->buffer_pkt, pkt);
-    } else {
-        int ret = av_packet_ref(ctx->internal->buffer_pkt, pkt);
-
-        if (ret < 0)
-            return ret;
-        av_packet_unref(pkt);
-    }
+    ret = av_packet_make_refcounted(pkt);
+    if (ret < 0)
+        return ret;
+    av_packet_move_ref(ctx->internal->buffer_pkt, pkt);
 
     return 0;
 }



More information about the ffmpeg-cvslog mailing list