[Home]

Summary:ASTERISK-12910: [patch] Inband DTMF on outbound call is not detected when dtmfmode=auto
Reporter:Makoto Dei (makoto)Labels:
Date Opened:2008-10-16 03:22:29Date Closed:2009-03-11 11:43:52
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_sip/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) dtmfmode-auto-outbound-call.patch
Description:Inband DTMF on outbound call is not detected when dtmfmode=auto
because p->vad (DSP for detecting inband DTMF) is not created.
On outbound call, p->vad is not created in sip_new(), so should be created
in process_sdp() when dtmfmode is switched to inband.

I have tested only on 1.2, but I believe that it happens on 1.4 or later.

Attached patch will fix the problem.
Comments:By: Leif Madsen (lmadsen) 2008-10-16 13:43:49

Would you mind verifying this on 1.4 for us? I will try to test this myself, but not sure when I will have the chance. Thanks!

By: Jason Parker (jparker) 2008-11-18 15:07:45.000-0600

Closing due to lack of response.

Please reopen once you are able to test on 1.4 or later.

By: Makoto Dei (makoto) 2009-02-24 22:55:20.000-0600

Sorry for lack of response.

I have tested this issue with 1.6.1-rc1 and reproduced it.

How to reproduce

1. Change dtmfmode from inband to auto in sip.conf
2. Call to the phone with Originate then execute VoiceMailMain like this:

Action: Originate
Channel: SIP/1001
Context: default
Exten: 1001
Priority: 1
Timeout: 30000
CallerID: 1001
Application: VoiceMailMain

3. Enter mailbox number, but asterisk doesn't detect DTMF


By: Digium Subversion (svnbot) 2009-03-11 11:36:51

Repository: asterisk
Revision: 181295

U   branches/1.4/channels/chan_sip.c

------------------------------------------------------------------------
r181295 | file | 2009-03-11 11:36:50 -0500 (Wed, 11 Mar 2009) | 9 lines

Fix a problem with inband DTMF detection on outgoing SIP calls when dtmfmode=auto.

When dtmfmode was set to auto the inband DTMF detector was not setup
on outgoing SIP calls. This caused inband DTMF detection to fail.
The inband DTMF detector is now setup for both dtmfmode inband and auto.

ssue ASTERISK-12910)
Reported by: makoto

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=181295

By: Digium Subversion (svnbot) 2009-03-11 11:37:57

Repository: asterisk
Revision: 181295

U   branches/1.4/channels/chan_sip.c

------------------------------------------------------------------------
r181295 | file | 2009-03-11 11:36:50 -0500 (Wed, 11 Mar 2009) | 9 lines

Fix a problem with inband DTMF detection on outgoing SIP calls when dtmfmode=auto.

When dtmfmode was set to auto the inband DTMF detector was not setup
on outgoing SIP calls. This caused inband DTMF detection to fail.
The inband DTMF detector is now setup for both dtmfmode inband and auto.

(closes issue ASTERISK-12910)
Reported by: makoto

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=181295

By: Digium Subversion (svnbot) 2009-03-11 11:40:49

Repository: asterisk
Revision: 181296

_U  trunk/
U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r181296 | file | 2009-03-11 11:40:48 -0500 (Wed, 11 Mar 2009) | 16 lines

Merged revisions 181295 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
 r181295 | file | 2009-03-11 13:36:50 -0300 (Wed, 11 Mar 2009) | 9 lines
 
 Fix a problem with inband DTMF detection on outgoing SIP calls when dtmfmode=auto.
 
 When dtmfmode was set to auto the inband DTMF detector was not setup
 on outgoing SIP calls. This caused inband DTMF detection to fail.
 The inband DTMF detector is now setup for both dtmfmode inband and auto.
 
 (closes issue ASTERISK-12910)
 Reported by: makoto
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=181296

By: Digium Subversion (svnbot) 2009-03-11 11:42:22

Repository: asterisk
Revision: 181297

_U  branches/1.6.0/
U   branches/1.6.0/channels/chan_sip.c

------------------------------------------------------------------------
r181297 | file | 2009-03-11 11:42:21 -0500 (Wed, 11 Mar 2009) | 23 lines

Merged revisions 181296 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
 r181296 | file | 2009-03-11 13:40:48 -0300 (Wed, 11 Mar 2009) | 16 lines
 
 Merged revisions 181295 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
   r181295 | file | 2009-03-11 13:36:50 -0300 (Wed, 11 Mar 2009) | 9 lines
   
   Fix a problem with inband DTMF detection on outgoing SIP calls when dtmfmode=auto.
   
   When dtmfmode was set to auto the inband DTMF detector was not setup
   on outgoing SIP calls. This caused inband DTMF detection to fail.
   The inband DTMF detector is now setup for both dtmfmode inband and auto.
   
   (closes issue ASTERISK-12910)
   Reported by: makoto
 ........
................

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=181297

By: Digium Subversion (svnbot) 2009-03-11 11:43:52

Repository: asterisk
Revision: 181298

_U  branches/1.6.1/
U   branches/1.6.1/channels/chan_sip.c

------------------------------------------------------------------------
r181298 | file | 2009-03-11 11:43:51 -0500 (Wed, 11 Mar 2009) | 23 lines

Merged revisions 181296 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
 r181296 | file | 2009-03-11 13:40:48 -0300 (Wed, 11 Mar 2009) | 16 lines
 
 Merged revisions 181295 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
   r181295 | file | 2009-03-11 13:36:50 -0300 (Wed, 11 Mar 2009) | 9 lines
   
   Fix a problem with inband DTMF detection on outgoing SIP calls when dtmfmode=auto.
   
   When dtmfmode was set to auto the inband DTMF detector was not setup
   on outgoing SIP calls. This caused inband DTMF detection to fail.
   The inband DTMF detector is now setup for both dtmfmode inband and auto.
   
   (closes issue ASTERISK-12910)
   Reported by: makoto
 ........
................

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=181298