From: Sean Bright <sean@seanbright.com>
Date: Mon, 1 Dec 2025 20:28:04 +0000 (-0500)
Subject: abstract_jb.c: Remove redundant timer check per static analysis.
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=af4d21d2914ee8634a84aea1cf47949f19e33f71;p=thirdparty%2Fasterisk.git

abstract_jb.c: Remove redundant timer check per static analysis.

While this check is technically unnecessary, it also was not harmful.

The 2 other items mentioned in the linked issue are false positives
and require no action.

Resolves: #1417
---

diff --git a/main/abstract_jb.c b/main/abstract_jb.c
index 03dd0226c0..6fda9dfb9d 100644
--- a/main/abstract_jb.c
+++ b/main/abstract_jb.c
@@ -966,7 +966,7 @@ static struct ast_frame *hook_event_cb(struct ast_channel *chan, struct ast_fram
 		return frame;
 	}
 
-	if (ast_channel_fdno(chan) == AST_JITTERBUFFER_FD && framedata->timer) {
+	if (ast_channel_fdno(chan) == AST_JITTERBUFFER_FD) {
 		if (ast_timer_ack(framedata->timer, 1) < 0) {
 			ast_log(LOG_ERROR, "Failed to acknowledge timer in jitter buffer\n");
 			return frame;