diff options
| author | Mark Thompson <sw@jkqxz.net> | 2018-02-11 00:51:33 +0000 |
|---|---|---|
| committer | Mark Thompson <sw@jkqxz.net> | 2018-02-20 22:04:12 +0000 |
| commit | ce5870a3a8f2b10668ee4f04c2ae0287f66f31b2 (patch) | |
| tree | 612022db516c9a96d051e9a55c89b503b99d5112 /libavcodec/h264_metadata_bsf.c | |
| parent | 13ca5d34ba5c473211daaae0a101123bcaada3e6 (diff) | |
cbs: Refcount all the things!
This makes it easier for users of the CBS API to get alloc/free right -
all subelements use the buffer API so that it's clear how to free them.
It also allows eliding some redundant copies: the packet -> fragment copy
disappears after this change if the input packet is refcounted, and more
codec-specific cases are now possible (but not included in this patch).
Diffstat (limited to 'libavcodec/h264_metadata_bsf.c')
| -rw-r--r-- | libavcodec/h264_metadata_bsf.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libavcodec/h264_metadata_bsf.c b/libavcodec/h264_metadata_bsf.c index 2b579e9d3d..6a1904e315 100644 --- a/libavcodec/h264_metadata_bsf.c +++ b/libavcodec/h264_metadata_bsf.c @@ -270,7 +270,7 @@ static int h264_metadata_filter(AVBSFContext *bsf, AVPacket *out) aud->primary_pic_type = j; err = ff_cbs_insert_unit_content(ctx->cbc, au, - 0, H264_NAL_AUD, aud); + 0, H264_NAL_AUD, aud, NULL); if (err < 0) { av_log(bsf, AV_LOG_ERROR, "Failed to insert AUD.\n"); goto fail; @@ -314,8 +314,8 @@ static int h264_metadata_filter(AVBSFContext *bsf, AVPacket *out) sei->nal_unit_header.nal_unit_type = H264_NAL_SEI; - err = ff_cbs_insert_unit_content(ctx->cbc, au, - sei_pos, H264_NAL_SEI, sei); + err = ff_cbs_insert_unit_content(ctx->cbc, au, sei_pos, + H264_NAL_SEI, sei, NULL); if (err < 0) { av_log(bsf, AV_LOG_ERROR, "Failed to insert SEI.\n"); goto fail; |