[FFmpeg-cvslog] pthread_frame: do not share priv_data between multiple codec contexts
Anton Khirnov
git at videolan.org
Fri Apr 10 17:53:18 EEST 2020
ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Sun Jan 15 14:52:04 2017 +0100| [e40107c1ad12a53bb7d81538f35b85ed3d11c4b0] | committer: Anton Khirnov
pthread_frame: do not share priv_data between multiple codec contexts
Specifically, between the user-facing one and the first frame thread
one.
This is fragile and dangerous, allocate separate private data for each
per-thread context.
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=e40107c1ad12a53bb7d81538f35b85ed3d11c4b0
---
libavcodec/pthread_frame.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
index 69671c90fb..1ad43d438a 100644
--- a/libavcodec/pthread_frame.c
+++ b/libavcodec/pthread_frame.c
@@ -702,7 +702,7 @@ void ff_frame_thread_free(AVCodecContext *avctx, int thread_count)
av_packet_unref(&p->avpkt);
av_freep(&p->released_buffers);
- if (i && p->avctx) {
+ if (p->avctx) {
if (codec->priv_class)
av_opt_free(p->avctx->priv_data);
av_freep(&p->avctx->priv_data);
@@ -812,7 +812,7 @@ int ff_frame_thread_init(AVCodecContext *avctx)
copy->internal->thread_ctx = p;
copy->internal->last_pkt_props = &p->avpkt;
- if (i) {
+ if (codec->priv_data_size) {
copy->priv_data = av_mallocz(codec->priv_data_size);
if (!copy->priv_data) {
err = AVERROR(ENOMEM);
@@ -825,9 +825,11 @@ int ff_frame_thread_init(AVCodecContext *avctx)
if (err < 0)
goto error;
}
- copy->internal->is_copy = 1;
}
+ if (i)
+ copy->internal->is_copy = 1;
+
if (codec->init)
err = codec->init(copy);
More information about the ffmpeg-cvslog
mailing list