From: Richard Mudgett <rmudgett@digium.com>
Date: Mon, 13 Dec 2010 16:56:07 +0000 (+0000)
Subject: Outgoing PRI/BRI calls cannot do DTMF triggered transfers.
X-Git-Tag: 1.4.40-rc1~30
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0543f25d7ca9fa347178c9661c87fe86a65f2d17;p=thirdparty%2Fasterisk.git

Outgoing PRI/BRI calls cannot do DTMF triggered transfers.

Outgoing PRI/BRI calls cannot do DTMF triggered transfers if a PROCEEDING
message is not received.  The debug output shows that the DTMF begin event
is seen, but the DTMF end event is missing.  When the DTMF begin happens,
the call is muted so we now have one way audio (until a DTMF end event is
somehow seen).

* Made set the proceeding flag when the PRI_EVENT_ANSWER event is
received.

* Made absorb the DTMF begin and DTMF end events if we are overlap dialing
and have not seen a PROCEEDING message.

* Added a debug message when absorbing a DTMF event.

JIRA SWP-2690
JIRA ABE-2697


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

diff --git a/channels/chan_dahdi.c b/channels/chan_dahdi.c
index fdf8cad3cf..c23127e8c5 100644
--- a/channels/chan_dahdi.c
+++ b/channels/chan_dahdi.c
@@ -5626,12 +5626,18 @@ static struct ast_frame  *dahdi_read(struct ast_channel *ast)
 					   a busy  */
 					f = NULL;
 				}
-			} else if (f->frametype == AST_FRAME_DTMF) {
+			} else if (f->frametype == AST_FRAME_DTMF_BEGIN
+				|| f->frametype == AST_FRAME_DTMF_END) {
 #ifdef HAVE_PRI
 				if (!p->proceeding && p->sig==SIG_PRI && p->pri && p->pri->overlapdial &&
 					((!p->outgoing && (p->pri->overlapdial & DAHDI_OVERLAPDIAL_INCOMING)) ||
 					(p->outgoing && (p->pri->overlapdial & DAHDI_OVERLAPDIAL_OUTGOING)))) {
 					/* Don't accept in-band DTMF when in overlap dial mode */
+					ast_log(LOG_DEBUG,
+						"Absorbing inband %s DTMF digit: 0x%02X '%c' on %s\n",
+						f->frametype == AST_FRAME_DTMF_BEGIN ? "begin" : "end",
+						f->subclass, f->subclass, ast->name);
+
 					f->frametype = AST_FRAME_NULL;
 					f->subclass = 0;
 				}
@@ -10100,6 +10106,7 @@ static void *pri_dchannel(void *vpri)
 							ast_log(LOG_DEBUG, "Waiting on answer confirmation on channel %d!\n", pri->pvts[chanpos]->channel);
 						} else {
 							pri->pvts[chanpos]->dialing = 0;
+							pri->pvts[chanpos]->proceeding = 1;
 							pri->pvts[chanpos]->subs[SUB_REAL].needanswer =1;
 							/* Enable echo cancellation if it's not on already */
 							dahdi_enable_ec(pri->pvts[chanpos]);