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

[Joshua Colp]

Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/2707

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/07/2707/1

diff --git a/apps/app_queue.c b/apps/app_queue.c
index a13af8b..3d7e6d5 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -10854,8 +10854,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/2707
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.8
Gerrit-Owner: Joshua Colp <jcolp at digium.com>