diff options
author | Zhao Zhili <zhilizhao@tencent.com> | 2024-06-30 18:45:00 +0800 |
---|---|---|
committer | Nuo Mi <nuomi2021@gmail.com> | 2024-07-11 20:24:11 +0800 |
commit | 54f9469fa173394ef3a19d8c6e700bca0b8fde81 (patch) | |
tree | 25de8aacde2ed24dcb88b3c917a8da8588f5f612 | |
parent | 1a86a7a48d60aaa64ff55c109ee64512576d7e8b (diff) | |
download | ffmpeg-54f9469fa173394ef3a19d8c6e700bca0b8fde81.tar.gz |
avutil/executor: Fix missing check before using mutex
-rw-r--r-- | libavutil/executor.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/libavutil/executor.c b/libavutil/executor.c index fb20104b58..89058fab2f 100644 --- a/libavutil/executor.c +++ b/libavutil/executor.c @@ -194,14 +194,17 @@ void av_executor_execute(AVExecutor *e, AVTask *t) AVTaskCallbacks *cb = &e->cb; AVTask **prev; - ff_mutex_lock(&e->lock); + if (e->thread_count) + ff_mutex_lock(&e->lock); if (t) { for (prev = &e->tasks; *prev && cb->priority_higher(*prev, t); prev = &(*prev)->next) /* nothing */; add_task(prev, t); } - ff_cond_signal(&e->cond); - ff_mutex_unlock(&e->lock); + if (e->thread_count) { + ff_cond_signal(&e->cond); + ff_mutex_unlock(&e->lock); + } if (!e->thread_count || !HAVE_THREADS) { // We are running in a single-threaded environment, so we must handle all tasks ourselves |