From 295b75b0942b640c54904065d4a9f9ccef78d239 Mon Sep 17 00:00:00 2001
From: Richard Fuchs <rfuchs@sipwise.com>
Date: Thu, 3 Oct 2024 11:25:19 -0400
Subject: [PATCH] MT#55283 extend lock duration

`next_exp_seq` must also be protected by the mutex.

Change-Id: I2fb223d9b29c891e45430887bfb3549d1a85b653
Warned-by: Coverity
---
 daemon/jitter_buffer.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/daemon/jitter_buffer.c b/daemon/jitter_buffer.c
index a6eae8b02..4647b86d2 100644
--- a/daemon/jitter_buffer.c
+++ b/daemon/jitter_buffer.c
@@ -355,8 +355,8 @@ static void set_jitter_values(struct media_packet *mp) {
 		if(rtp_pt->codec_def && rtp_pt->codec_def->dtmf)
 			dtmf = 1;
 	}
+	mutex_lock(&jb->lock);
 	if(jb->next_exp_seq && !dtmf) {
-		mutex_lock(&jb->lock);
 		if(curr_seq > jb->next_exp_seq) {
 			int marker = (mp->rtp->m_pt & 0x80) ? 1 : 0;
 			if(!marker) {
@@ -382,10 +382,10 @@ static void set_jitter_values(struct media_packet *mp) {
 
 		if(jb->cont_miss >= CONT_MISS_COUNT)
 			reset_jitter_buffer(jb);
-		mutex_unlock(&jb->lock);
 	}
 	if(curr_seq >= jb->next_exp_seq)
 		jb->next_exp_seq = curr_seq + 1;
+	mutex_unlock(&jb->lock);
 }
 
 static void __jb_send_later(struct timerthread_queue *ttq, void *p) {