Summary:	ASTERISK-10038: Repeated DTMF
Reporter:	Steve Feinstein (stevefeinstein)	Labels:
Date Opened:	2007-08-07 13:50:19	Date Closed:	2007-08-10 09:55:12
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Applications/app_meetme
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) meetme_dtmf.conf
Description:	When setting up a meetme conference with DTMF passthrough (the 'F' option) and the 'X' option and an appropriate MEETME_EXIT_CONTEXT, when DTMF is sent to the conference by the marked participant, the DTMF it will repeat, sending many tens, or hundreds of DTMF See attached sample dialplan for example of how to reproduce.   Configure extension 'dott' to dial out through PSTN This will connect the caller to the destination number through meetme. Then press a DTMF digit, and you should hear it start but not stop on the called (labeled 'one') phone. ****** ADDITIONAL INFORMATION ****** Problem also happens in trunk.
Comments:	By: Digium Subversion (svnbot) 2007-08-09 10:55:19 Repository: asterisk Revision: 78717 ------------------------------------------------------------------------ r78717 | russell | 2007-08-09 10:55:18 -0500 (Thu, 09 Aug 2007) | 7 lines Fix a problem with the combination of the 'F' option to pass DTMF through a conference and options that use DTMF to activate various features.  The problem was that the BEGIN frame would be passed through, but the END frame would get intercepted to activate a feature.  Then, the other conference members would hear DTMF for forever, which they didn't seem to like very much. (closes issue ASTERISK-10038, reported by stevefeinstein, fixed by me) ------------------------------------------------------------------------ By: Digium Subversion (svnbot) 2007-08-09 10:55:49 Repository: asterisk Revision: 78718 ------------------------------------------------------------------------ r78718 | russell | 2007-08-09 10:55:48 -0500 (Thu, 09 Aug 2007) | 15 lines Merged revisions 78717 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78717 | russell | 2007-08-09 11:12:57 -0500 (Thu, 09 Aug 2007) | 7 lines Fix a problem with the combination of the 'F' option to pass DTMF through a conference and options that use DTMF to activate various features.  The problem was that the BEGIN frame would be passed through, but the END frame would get intercepted to activate a feature.  Then, the other conference members would hear DTMF for forever, which they didn't seem to like very much. (closes issue ASTERISK-10038, reported by stevefeinstein, fixed by me) ........ ------------------------------------------------------------------------ By: Russell Bryant (russell) 2007-08-09 10:57:08 Thank you for the detailed report! By: Digium Subversion (svnbot) 2007-08-10 09:55:12 Repository: asterisk Revision: 78994 ------------------------------------------------------------------------ r78994 | murf | 2007-08-10 09:55:07 -0500 (Fri, 10 Aug 2007) | 117 lines Merged revisions 78570,78590,78635,78637,78648-78649,78679,78683,78685-78686,78718,78747 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r78570 | file | 2007-08-08 07:52:13 -0600 (Wed, 08 Aug 2007) | 12 lines Merged revisions 78569 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78569 | file | 2007-08-08 10:51:01 -0300 (Wed, 08 Aug 2007) | 4 lines (closes issue ASTERISK-9984) Reported by: adamgundy Update sip.conf to include another scenario where directrtpsetup will fail. ........ ................ r78590 | mmichelson | 2007-08-08 08:34:10 -0600 (Wed, 08 Aug 2007) | 12 lines Merged revisions 78575 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78575 | mmichelson | 2007-08-08 09:26:36 -0500 (Wed, 08 Aug 2007) | 4 lines Changing a bit of logic so that someone will NEVER exit the queue on timeout unless they have enabled the 'n' option. This commit relates to issue ASTERISK-9971. Thanks to jfitzgibbon for detailing the idea behind this code change. ........ ................ r78635 | mmichelson | 2007-08-08 12:34:16 -0600 (Wed, 08 Aug 2007) | 11 lines Blocked revisions 78620 via svnmerge ........ r78620 | mmichelson | 2007-08-08 13:16:49 -0500 (Wed, 08 Aug 2007) | 4 lines Fixed some compiler warnings so that compiling with dev-mode and IMAP storage would not have any errors. This section of code may get changed again shortly since my change uncovers a rather silly bit of logic. ........ ................ r78637 | file | 2007-08-08 13:03:46 -0600 (Wed, 08 Aug 2007) | 2 lines Correct spelling. s/threaads/threads/ ................ r78648 | qwell | 2007-08-08 13:30:33 -0600 (Wed, 08 Aug 2007) | 10 lines Merged revisions 78646 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78646 | qwell | 2007-08-08 14:29:42 -0500 (Wed, 08 Aug 2007) | 2 lines Fix mogs email address. ........ ................ r78649 | file | 2007-08-08 13:30:52 -0600 (Wed, 08 Aug 2007) | 2 lines Merge audiohooks branch into trunk. This is a new API for developers to listen and manipulate the audio going through a channel. ................ r78679 | file | 2007-08-08 14:49:07 -0600 (Wed, 08 Aug 2007) | 2 lines HAVEL_SS7 should be HAVE_SS7. Reported by kwallace. ................ r78683 | file | 2007-08-08 15:44:58 -0600 (Wed, 08 Aug 2007) | 2 lines Add support for using epoll instead of poll. This should increase scalability and is done in such a way that we should be able to add support for other poll() replacements. ................ r78685 | file | 2007-08-08 15:58:12 -0600 (Wed, 08 Aug 2007) | 2 lines Minor fix for building under dev mode when byteswapping macro header files are not available. ................ r78686 | file | 2007-08-08 16:05:45 -0600 (Wed, 08 Aug 2007) | 2 lines Regenerate configure script. This actually just updated the revision number... since my last merge changed it to an older number, while it was in fact generated from a much newer revision. ................ r78718 | russell | 2007-08-09 10:13:26 -0600 (Thu, 09 Aug 2007) | 15 lines Merged revisions 78717 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78717 | russell | 2007-08-09 11:12:57 -0500 (Thu, 09 Aug 2007) | 7 lines Fix a problem with the combination of the 'F' option to pass DTMF through a conference and options that use DTMF to activate various features.  The problem was that the BEGIN frame would be passed through, but the END frame would get intercepted to activate a feature.  Then, the other conference members would hear DTMF for forever, which they didn't seem to like very much. (closes issue ASTERISK-10038, reported by stevefeinstein, fixed by me) ........ ................ r78747 | russell | 2007-08-09 11:07:36 -0600 (Thu, 09 Aug 2007) | 4 lines Fix a problem that I had introduced into MWI handling.  I had ignored the mailbox context.  Now, all related MWI event dealings pay attention to the context as well. ................ ------------------------------------------------------------------------