diff options
author | James Almer <jamrial@gmail.com> | 2017-10-01 23:31:12 -0300 |
---|---|---|
committer | James Almer <jamrial@gmail.com> | 2017-10-02 15:04:40 -0300 |
commit | 0c1ffd0aa55c6cef6dffe2b736786c6cb86d8a3d (patch) | |
tree | 49b3fb8876d1a46f8ac93490c4087dcc7f9f38ee /libavdevice | |
parent | a22c6a4796ca1f2cbee6784262515da876fbec22 (diff) | |
download | ffmpeg-streaming-0c1ffd0aa55c6cef6dffe2b736786c6cb86d8a3d.zip ffmpeg-streaming-0c1ffd0aa55c6cef6dffe2b736786c6cb86d8a3d.tar.gz |
avdevice/decklink_dec: use av_packet_add_side_data()
It uses the existing buffer instead of allocating a new one.
Reviewed-by: Marton Balint <cus@passwd.hu>
Signed-off-by: James Almer <jamrial@gmail.com>
Diffstat (limited to 'libavdevice')
-rw-r--r-- | libavdevice/decklink_dec.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp index 8a14094..9d12d0f 100644 --- a/libavdevice/decklink_dec.cpp +++ b/libavdevice/decklink_dec.cpp @@ -390,10 +390,8 @@ uint8_t *get_metadata(AVFormatContext *avctx, uint16_t *buf, size_t width, clear_parity_bits(buf, len); data = vanc_to_cc(avctx, buf, width, data_len); if (data) { - uint8_t *pkt_cc = av_packet_new_side_data(pkt, AV_PKT_DATA_A53_CC, data_len); - if (pkt_cc) - memcpy(pkt_cc, data, data_len); - av_free(data); + if (av_packet_add_side_data(pkt, AV_PKT_DATA_A53_CC, data, data_len) < 0) + av_free(data); } } else { av_log(avctx, AV_LOG_DEBUG, "Unknown meta data DID = 0x%.2x SDID = 0x%.2x\n", |