[Asterisk-code-review] app queue: Fix crash when unloading module. (asterisk[certified/13.1])

Joshua Colp asteriskteam at digium.com
Tue Apr 26 06:15:47 CDT 2016


Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/2706

Change subject: app_queue: Fix crash when unloading module.
......................................................................

app_queue: Fix crash when unloading module.

When unloading the app_queue module the members in each queue are
destroyed and as part of this they are removed from the pending
members container. Unfortunately a crash would occur as the container
was destroyed before the members were removed.

This change tweaks ordering so the container destruction occurs
after the members are destroyed.

ASTERISK-16115

Change-Id: I48c728668c55aee3d05b751a5d450fb57e87f44b
---
M apps/app_queue.c
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/06/2706/1

diff --git a/apps/app_queue.c b/apps/app_queue.c
index 972ca9f..e490278 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -10704,8 +10704,9 @@
 	ast_extension_state_del(0, extension_state_cb);
 
 	ast_unload_realtime("queue_members");
-	ao2_cleanup(pending_members);
 	ao2_cleanup(queues);
+	ao2_cleanup(pending_members);
+
 	queues = NULL;
 	return 0;
 }

-- 
To view, visit https://gerrit.asterisk.org/2706
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I48c728668c55aee3d05b751a5d450fb57e87f44b
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: certified/13.1
Gerrit-Owner: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list