From 2ecdc84d079ab34338d2cdfcefb6eae8694e06b6 Mon Sep 17 00:00:00 2001 From: Sean Bright Date: Mon, 1 Dec 2025 15:28:04 -0500 Subject: [PATCH] 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 --- main/abstract_jb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; -- 2.47.3