diff options
| author | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-07-16 20:23:02 +0000 |
|---|---|---|
| committer | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-07-16 20:23:02 +0000 |
| commit | 532dac8497cdba9895479ccca131922a161659a8 (patch) | |
| tree | b03ffd3dcc3f6f99af8be80c632a463c3bba4464 /apps/app_queue.c | |
| parent | a52eadf50649102ee66d8d832227b216331826ba (diff) | |
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: http://svn.digium.com/svn/asterisk/branches/1.4@131369 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'apps/app_queue.c')
| -rw-r--r-- | apps/app_queue.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/apps/app_queue.c b/apps/app_queue.c index 113826296..1ddfc9fb5 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) { |