Merged revisions 43916 via svnmerge from

https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r43916 | kpfleming | 2006-09-28 12:31:57 -0500 (Thu, 28 Sep 2006) | 2 lines

fix buggy (and overly complex) loop used during reload of app_queue for static member list updating

........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@43919 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Kevin P. Fleming
2006-09-28 17:35:42 +00:00
parent 13b9f34966
commit 78e8232ef8

View File

@@ -3708,7 +3708,7 @@ static int reload_queues(void)
struct ast_config *cfg;
char *cat, *tmp;
struct ast_variable *var;
struct member *prev, *cur, *newm;
struct member *prev, *cur, *newm, *next;
int new;
const char *general_val = NULL;
char parse[80];
@@ -3825,22 +3825,19 @@ static int reload_queues(void)
}
/* Free remaining members marked as delme */
for (prev = NULL, newm = NULL, cur = q->members; cur; prev = cur, cur = cur->next) {
if (newm) {
free(newm);
newm = NULL;
}
for (prev = NULL, cur = q->members, next = cur ? cur->next : NULL;
cur;
cur = next, next = cur ? cur->next : NULL) {
if (!cur->delme)
continue;
if (cur->delme) {
if (prev) {
prev->next = cur->next;
newm = cur;
} else {
q->members = cur->next;
newm = cur;
}
remove_from_interfaces(cur->interface);
}
if (prev)
prev->next = next;
else
q->members = next;
remove_from_interfaces(cur->interface);
free(cur);
}
if (q->strategy == QUEUE_STRATEGY_ROUNDROBIN)