diff options
author | murf <murf@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-06-16 20:43:46 +0000 |
---|---|---|
committer | murf <murf@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-06-16 20:43:46 +0000 |
commit | 5e1dbbb4068d8b761794fbfd02d9adb2687b2740 (patch) | |
tree | 5a167c96702022ee82db6e25d50ebef659661b6f /apps/app_queue.c | |
parent | 0b9086d8bbdb25af369dfc899d2617edb6bcf3c3 (diff) |
(closes issue #12689)
Reported by: ys
Many thanks to ys for doing the research on this problem.
I didn't think it would be best to unlock the contexts
and then relock them after the remove_extension2() call,
so I added an extra arg to remove_extension2() and set it
appropriately in each call. There were not that many.
I considered forcing the code to lock the contexts before
the call to remove_extension2(), but that would require
a slightly greater degree of changes, especially since
the find_context_locked is local to pbx.c
I did a simple sanity test to make sure the code doesn't
mess things up in general.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@123165 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 5e7289ae0..6d40c3976 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -6366,7 +6366,7 @@ static int unload_module(void) ast_event_unsubscribe(device_state_sub); if ((con = ast_context_find("app_queue_gosub_virtual_context"))) { - ast_context_remove_extension2(con, "s", 1, NULL); + ast_context_remove_extension2(con, "s", 1, NULL, 0); ast_context_destroy(con, "app_queue"); /* leave no trace */ } |