[asterisk-commits] rmudgett: branch 1.8 r301134 - /branches/1.8/channels/chan_dahdi.c

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Fri Jan  7 19:11:31 2011
New Revision: 301134

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=301134
Log:
The DTMF attended transfer feature cannot callback a chan_dahdi BRI phone.

The DAHDI ISDN channel name is not dialable.

Make a channel name like DAHDI/i3/400-12 dialable when the sequence number
is stripped off of the name.

Modified:
    branches/1.8/channels/chan_dahdi.c

Modified: branches/1.8/channels/chan_dahdi.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/channels/chan_dahdi.c?view=diff&rev=301134&r1=301133&r2=301134
==============================================================================
--- branches/1.8/channels/chan_dahdi.c (original)
+++ branches/1.8/channels/chan_dahdi.c Fri Jan  7 19:11:31 2011
@@ -12773,6 +12773,11 @@
 		if (!p->pri || p->pri->span != span) {
 			return 0;
 		}
+		if (!groupmatch && channelmatch == -1) {
+			/* Match any group since it only needs to be on the PRI span. */
+			*groupmatched = 1;
+			return 1;
+		}
 	}
 #endif	/* defined(HAVE_PRI) */
 	/* check group matching */
@@ -13083,11 +13088,14 @@
 	 * data is ---v
 	 * Dial(DAHDI/pseudo[/extension[/options]])
 	 * Dial(DAHDI/<channel#>[c|r<cadance#>|d][/extension[/options]])
+	 * Dial(DAHDI/<subdir>!<channel#>[c|r<cadance#>|d][/extension[/options]])
+	 * Dial(DAHDI/i<span>[/extension[/options]])
 	 * Dial(DAHDI/[i<span>-](g|G|r|R)<group#(0-63)>[c|r<cadance#>|d][/extension[/options]])
-	 * Dial(DAHDI/<subdir>!<channel#>[c|r<cadance#>|d][/extension[/options]])
 	 *
 	 * i - ISDN span channel restriction.
 	 *     Used by CC to ensure that the CC recall goes out the same span.
+	 *     Also to make ISDN channel names dialable when the sequence number
+	 *     is stripped off.  (Used by DTMF attended transfer feature.)
 	 *
 	 * g - channel group allocation search forward
 	 * G - channel group allocation search backward
@@ -13136,8 +13144,8 @@
 		/* Remove the ISDN span channel restriction specifier. */
 		s = strchr(args.group, '-');
 		if (!s) {
-			ast_log(LOG_WARNING, "Bad ISDN span format for data %s\n", data);
-			return NULL;
+			/* Search all groups since we are ISDN span restricted. */
+			return iflist;
 		}
 		args.group = s + 1;
 		res = 0;