From: Joshua Colp <jcolp@digium.com>
Date: Fri, 6 Feb 2009 19:28:53 +0000 (+0000)
Subject: Don't subscribe to a mailbox on pseudo channels. It is futile. This solves an issue... 
X-Git-Tag: 1.6.2.0-beta1~278
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=fa389ab17acf4ae9181b87feb7890000f87be9df;p=thirdparty%2Fasterisk.git

Don't subscribe to a mailbox on pseudo channels. It is futile. This solves an issue where duplicated pseudo channels would cause a crash because the first one would unsubscribe and the next one would also try to unsubscribe the same subscription.
(closes issue #14322)
Reported by: amessina


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@174041 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/channels/chan_dahdi.c b/channels/chan_dahdi.c
index c783fb85db..db2f842aaf 100644
--- a/channels/chan_dahdi.c
+++ b/channels/chan_dahdi.c
@@ -9383,7 +9383,7 @@ static struct dahdi_pvt *mkintf(int channel, const struct dahdi_chan_conf *conf,
 		tmp->cid_ton = 0;
 		ast_copy_string(tmp->cid_name, conf->chan.cid_name, sizeof(tmp->cid_name));
 		ast_copy_string(tmp->mailbox, conf->chan.mailbox, sizeof(tmp->mailbox));
-		if (!ast_strlen_zero(tmp->mailbox)) {
+		if (channel != CHAN_PSEUDO && !ast_strlen_zero(tmp->mailbox)) {
 			char *mailbox, *context;
 			mailbox = context = ast_strdupa(tmp->mailbox);
 			strsep(&context, "@");