diff options
author | Luca Barbato <lu_zero@gentoo.org> | 2015-10-23 11:11:33 +0200 |
---|---|---|
committer | Luca Barbato <lu_zero@gentoo.org> | 2015-10-26 18:00:55 +0100 |
commit | 9b56d5c11488656254e9aed8d964ef2b7c2ff5e6 (patch) | |
tree | 83fe8dc6bd2410a174d214400dcf1dca290e7f49 /libavformat | |
parent | d584533cf38141172e20bae5436629ee17c8ce50 (diff) |
avpacket: Deprecate av_dup_packet
As documented, `av_dup_packet` is broken by design, `av_packet_ref`
matches the AVFrame ref-counted API and can be safely used instead.
Diffstat (limited to 'libavformat')
-rw-r--r-- | libavformat/matroskaenc.c | 7 | ||||
-rw-r--r-- | libavformat/mux.c | 10 |
2 files changed, 5 insertions, 12 deletions
diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index f1bd272672..baffd54340 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -1608,12 +1608,7 @@ static int mkv_write_packet(AVFormatContext *s, AVPacket *pkt) // buffer an audio packet to ensure the packet containing the video // keyframe's timecode is contained in the same cluster for WebM if (codec_type == AVMEDIA_TYPE_AUDIO) { - mkv->cur_audio_pkt = *pkt; - if (pkt->buf) { - mkv->cur_audio_pkt.buf = av_buffer_ref(pkt->buf); - ret = mkv->cur_audio_pkt.buf ? 0 : AVERROR(ENOMEM); - } else - ret = av_dup_packet(&mkv->cur_audio_pkt); + ret = av_packet_ref(&mkv->cur_audio_pkt, pkt); } else ret = mkv_write_packet_internal(s, pkt); return ret; diff --git a/libavformat/mux.c b/libavformat/mux.c index e86d202713..4a81e36af5 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -416,12 +416,8 @@ int ff_interleave_add_packet(AVFormatContext *s, AVPacket *pkt, this_pktl = av_mallocz(sizeof(AVPacketList)); if (!this_pktl) return AVERROR(ENOMEM); - this_pktl->pkt = *pkt; - pkt->buf = NULL; - pkt->side_data = NULL; - pkt->side_data_elems = 0; - // Duplicate the packet if it uses non-allocated memory - if ((ret = av_dup_packet(&this_pktl->pkt)) < 0) { + + if ((ret = av_packet_ref(&this_pktl->pkt, pkt)) < 0) { av_free(this_pktl); return ret; } @@ -450,6 +446,8 @@ next_non_null: s->streams[pkt->stream_index]->last_in_packet_buffer = *next_point = this_pktl; + av_packet_unref(pkt); + return 0; } |