Commit 2ecdc84d07 for asterisk.org
commit 2ecdc84d079ab34338d2cdfcefb6eae8694e06b6
Author: Sean Bright <sean@seanbright.com>
Date: Mon Dec 1 15:28:04 2025 -0500
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;