On Mon, 2 Oct 2017, James Almer wrote:

Signed-off-by: James Almer <jamr...@gmail.com>
---
Untested.

Tested and works.


libavdevice/decklink_dec.cpp | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp
index 8a14094474..d8c624aa5d 100644
--- a/libavdevice/decklink_dec.cpp
+++ b/libavdevice/decklink_dec.cpp
@@ -458,16 +458,15 @@ static int avpacket_queue_put(AVPacketQueue *q, AVPacket 
*pkt)
        av_log(q->avctx, AV_LOG_WARNING,  "Decklink input buffer overrun!\n");
        return -1;
    }
-    /* duplicate the packet */
-    if (av_dup_packet(pkt) < 0) {
-        return -1;
-    }

-    pkt1 = (AVPacketList *)av_malloc(sizeof(AVPacketList));
+    pkt1 = (AVPacketList *)av_mallocz(sizeof(AVPacketList));
    if (!pkt1) {
        return -1;
    }
-    pkt1->pkt  = *pkt;
+    if (av_packet_ref(&pkt1->pkt, pkt) < 0) {
+        return -1;
+    }
+    av_packet_unref(pkt);
    pkt1->next = NULL;

    pthread_mutex_lock(&q->mutex);

LGTM, thanks.

Marton
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to