kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2026-06-10 04:26:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

app_queue: Fix rN raise_penalty ignoring min_penalty in calc_metric

Browse Source 
QUEUE_RAISE_PENALTY=rN was not respected during member selection. calc_metric() raised penalties below QUEUE_MIN_PENALTY, allowing excluded members to be selected.

This change makes calc_metric() honor raise_respect_min, keeping behavior consistent with queue empty checks and expected rN semantics

UserNote: Fixes an issue where QUEUE_RAISE_PENALTY=rN could raise a member’s penalty below QUEUE_MIN_PENALTY during member selection. This could allow members intended to be excluded to be selected. The queue now consistently respects the minimum penalty when raising penalties, aligning member selection behavior with queue empty checks and documented rN semantics.
This commit is contained in:
phoneben
2026-01-06 21:53:45 +02:00
committed by Asterisk Development Team
parent e9a41c6582
commit e9fc5b041f
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
apps/app_queue.c
View File

@@ -6290,7 +6290,7 @@ static int calc_metric(struct call_queue *q, struct member *mem, int pos, struct
int penalty = mem->penalty;
if (usepenalty) {
if (qe->raise_penalty != INT_MAX && penalty < qe->raise_penalty) {
if (qe->raise_penalty != INT_MAX && penalty < qe->raise_penalty && !(qe->raise_respect_min && qe->min_penalty != INT_MAX && penalty < qe->min_penalty)) {
/* Low penalty is raised up to the current minimum */
penalty = qe->raise_penalty;
}