[Asterisk-code-review] pbx dundi: reordered unloading of module pbx dundi (asterisk[master])

Jenkins2 asteriskteam at digium.com
Fri Jun 29 15:27:21 CDT 2018


Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/9312 )

Change subject: pbx_dundi: reordered unloading of module pbx_dundi
......................................................................

pbx_dundi: reordered unloading of module pbx_dundi

Destroy scheduler after peers are pruned to stop dundi crashing when
unloading module.

ASTERISK-26987
Reported-by: Kirsty Tyerman

Change-Id: Ic12e562cd90d8d813a9e97f302045091f59e3c05
---
M pbx/pbx_dundi.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  Richard Mudgett: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/pbx/pbx_dundi.c b/pbx/pbx_dundi.c
index e356d6f..a94d71c 100644
--- a/pbx/pbx_dundi.c
+++ b/pbx/pbx_dundi.c
@@ -4982,13 +4982,14 @@
 
 	close(netsocket);
 	io_context_destroy(io);
-	ast_sched_context_destroy(sched);
 
 	mark_mappings();
 	prune_mappings();
 	mark_peers();
 	prune_peers();
 
+	ast_sched_context_destroy(sched);
+
 	return 0;
 }
 

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic12e562cd90d8d813a9e97f302045091f59e3c05
Gerrit-Change-Number: 9312
Gerrit-PatchSet: 1
Gerrit-Owner: Kirsty Tyerman <kirsty.tyerman at boeing.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180629/7485b397/attachment-0001.html>


More information about the asterisk-code-review mailing list