[Asterisk-code-review] chan iax2: On reload make sure to check for existing MWI sub... (asterisk[master])

[Sergej Kasumovic]

Sergej Kasumovic has uploaded this change for review. ( https://gerrit.asterisk.org/6019


Change subject: chan_iax2: On reload make sure to check for existing MWI subscription
......................................................................

chan_iax2: On reload make sure to check for existing MWI subscription

On every reload of chan_iax2 module, MWI subscription was added, which
results in additional taskprocessors being accumulated over time.

This commit fixes it by making sure we check for existing subscription
first.

This was verified with 'core show taskprocessors' CLI command.

ASTERISK-27122 #close

Change-Id: Ie2ef528fd5ca01b933eeb88188cc10967899cfb9
---
M channels/chan_iax2.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/19/6019/1

diff --git a/channels/chan_iax2.c b/channels/chan_iax2.c
index f40873e..5abb6c3 100644
--- a/channels/chan_iax2.c
+++ b/channels/chan_iax2.c
@@ -13055,7 +13055,7 @@
 		ast_free_acl_list(oldacl);
 	}
 
-	if (!ast_strlen_zero(peer->mailbox)) {
+	if (!ast_strlen_zero(peer->mailbox) && !peer->mwi_event_sub) {
 		struct stasis_topic *mailbox_specific_topic;
 
 		mailbox_specific_topic = ast_mwi_topic(peer->mailbox);

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie2ef528fd5ca01b933eeb88188cc10967899cfb9
Gerrit-Change-Number: 6019
Gerrit-PatchSet: 1
Gerrit-Owner: Sergej Kasumovic <sergej at bicomsystems.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170714/bd3e07bf/attachment.html>