diff mbox series

[FFmpeg-devel] fftools/ffmpeg_sched: actually initialize/destroy schedule_lock

Message ID 20231214091748.31332-1-anton@khirnov.net
State Accepted
Commit 06d5dc11db257786dd18d13f39dd063ef8c1fa7d
Headers show
Series [FFmpeg-devel] fftools/ffmpeg_sched: actually initialize/destroy schedule_lock | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Anton Khirnov Dec. 14, 2023, 9:17 a.m. UTC 
---
 fftools/ffmpeg_sched.c | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Thilo Borgmann Dec. 14, 2023, 10:20 a.m. UTC | #1 
Am 14.12.23 um 10:17 schrieb Anton Khirnov:
> ---
>   fftools/ffmpeg_sched.c | 6 ++++++
>   1 file changed, 6 insertions(+)

Fixes --assert-level=2 for me. LGTM.

-Thilo
diff mbox series

Patch

diff --git a/fftools/ffmpeg_sched.c b/fftools/ffmpeg_sched.c
index 96eaaae072..20063fe47e 100644
--- a/fftools/ffmpeg_sched.c
+++ b/fftools/ffmpeg_sched.c
@@ -565,6 +565,8 @@  void sch_free(Scheduler **psch)
 
     av_freep(&sch->sdp_filename);
 
+    pthread_mutex_destroy(&sch->schedule_lock);
+
     pthread_mutex_destroy(&sch->mux_ready_lock);
 
     pthread_mutex_destroy(&sch->mux_done_lock);
@@ -590,6 +592,10 @@  Scheduler *sch_alloc(void)
     sch->class    = &scheduler_class;
     sch->sdp_auto = 1;
 
+    ret = pthread_mutex_init(&sch->schedule_lock, NULL);
+    if (ret)
+        goto fail;
+
     ret = pthread_mutex_init(&sch->mux_ready_lock, NULL);
     if (ret)
         goto fail;