From 592497392bf8317a1f60c7bdd0653218af6ca6a0 Mon Sep 17 00:00:00 2001 From: Mark Michelson Date: Wed, 16 Jul 2008 20:23:02 +0000 Subject: [PATCH] Move the init_queue call back to where it used to be (changed Sept 12 last year). It was moved then to prevent a memory leak. Since then, the same memory leak recurred and was fixed in a better way. Now it has been found that the placement of this init_queue call can cause problems if a realtime queue has values changed to an empty string. The problem is that the default value for that queue parameter would not be set. (closes issue #13084) Reported by: elbriga git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@131369 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- apps/app_queue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/app_queue.c b/apps/app_queue.c index 1138262969..1ddfc9fb5f 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -1229,9 +1229,9 @@ static struct call_queue *find_queue_by_name_rt(const char *queuename, struct as ast_mutex_lock(&q->lock); clear_queue(q); q->realtime = 1; - init_queue(q); /* Ensure defaults for all parameters not set explicitly. */ AST_LIST_INSERT_HEAD(&queues, q, list); } + init_queue(q); /* Ensure defaults for all parameters not set explicitly. */ memset(tmpbuf, 0, sizeof(tmpbuf)); for (v = queue_vars; v; v = v->next) {