Asterisk
  1. Asterisk
  2. ASTERISK-15021

[patch] Auto-fallthrough when attempting to enter DTMF using Background() in a Macro()

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Component/s: General
    • Labels:
      None
    • Mantis ID:
      16113
    • Regression:
      Yes

      Description

      Hello, All.

      Asterisk calls me using a call-file, then i make call with the following macro:

      [macro-internalcall-with-redirect-callback]
      exten => s,1,Dial(SIP/$

      {ARG1}

      ,20,jgtedH)
      exten => s,n,Goto(s-$

      {DIALSTATUS}

      ,1)
      exten => s-BUSY,1,Background(busy-cycl0)
      exten => s-BUSY,2,Goto($

      {CONTEXT},s,1)

      In ${MACRO_CONTEXT} i have extensions:
      exten => *,1,ResetCDR(w)
      exten => *,n,Goto(${CONTEXT}

      ,s,5)

      I make a call to _XXX, recieve "BUSY". While busy-cycl0 is playing, i hit the "*"-key.

      1.6.1.7rc2 - Strange behavior:
      [Oct 16 18:03:19] – Executing [s@macro-internalcall-with-redirect-callback:3] Goto("H323/sibirtelecom-1", "s-BUSY,1") in new stack
      [Oct 16 18:03:19] – Goto (macro-internalcall-with-redirect-callback,s-BUSY,1)
      [Oct 16 18:03:19] – Executing [s-BUSY@macro-internalcall-with-redirect-callback:1] BackGround("H323/sibirtelecom-1", "busy-cycl0") in new stack
      [Oct 16 18:03:19] – <H323/sibirtelecom-1> Playing 'busy-cycl0.alaw' (language 'en')
      [Oct 16 18:03:20] – Auto fallthrough, channel 'H323/sibirtelecom-1' status is 'BUSY'
      [Oct 16 18:03:24] DEBUG[9435]: pbx_spool.c:405 scan_service: Delaying retry since we're currently running '/var/spool/asterisk/outgoing/32631.call'
      [Oct 16 18:03:27] NOTICE[9475]: pbx_spool.c:348 attempt_thread: Call completed to Local/2487305@qqq

      1.6.1.4 - Normal behavior:
      [Oct 16 18:06:43] – Executing [s@macro-internalcall-with-redirect-callback:3] Goto("H323/sibirtelecom-1", "s-BUSY,1") in new stack
      [Oct 16 18:06:43] – Goto (macro-internalcall-with-redirect-callback,s-BUSY,1)
      [Oct 16 18:06:43] – Executing [s-BUSY@macro-internalcall-with-redirect-callback:1] BackGround("H323/sibirtelecom-1", "busy-cycl0") in new stack
      [Oct 16 18:06:43] – <H323/sibirtelecom-1> Playing 'busy-cycl0.alaw' (language 'en')
      [Oct 16 18:06:44] == CDR updated on H323/sibirtelecom-1
      [Oct 16 18:06:44] – Executing [*@callback:1] ResetCDR("H323/sibirtelecom-1", "w") in new stack
      [Oct 16 18:06:44] – Executing [*@callback:2] Goto("H323/sibirtelecom-1", "callback,s,5") in new stack
      [Oct 16 18:06:44] – Goto (callback,s,5)
      [Oct 16 18:06:44] – Executing [s@callback:5] WaitExten("H323/sibirtelecom-1", "1") in new stack

      This error occurs both with chan_h323 and with chan_ooh323.

      P.S. I don't know, what category should i select in this case, please move the topic to the correct one.

        Issue Links

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

          Hide
          Digium Subversion added a comment -

          Repository: asterisk
          Revision: 235421

          U branches/1.4/main/pbx.c

          ------------------------------------------------------------------------
          r235421 | tilghman | 2009-12-17 11:17:52 -0600 (Thu, 17 Dec 2009) | 8 lines

          Use context from which Macro is executed, not macro context, if applicable.
          Also, ensure that the extension COULD match, not just that it won't match more.
          (closes issue ASTERISK-15021)
          Reported by: OrNix
          Patches:
          20091216__issue16113.diff.txt uploaded by tilghman (license 14)
          Tested by: OrNix

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

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

          Show
          Digium Subversion added a comment - Repository: asterisk Revision: 235421 U branches/1.4/main/pbx.c ------------------------------------------------------------------------ r235421 | tilghman | 2009-12-17 11:17:52 -0600 (Thu, 17 Dec 2009) | 8 lines Use context from which Macro is executed, not macro context, if applicable. Also, ensure that the extension COULD match, not just that it won't match more. (closes issue ASTERISK-15021 ) Reported by: OrNix Patches: 20091216__issue16113.diff.txt uploaded by tilghman (license 14) Tested by: OrNix ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=235421
          Hide
          Digium Subversion added a comment -

          Repository: asterisk
          Revision: 235422

          _U trunk/
          U trunk/main/pbx.c

          ------------------------------------------------------------------------
          r235422 | tilghman | 2009-12-17 11:19:08 -0600 (Thu, 17 Dec 2009) | 15 lines

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

          ........
          r235421 | tilghman | 2009-12-17 11:17:51 -0600 (Thu, 17 Dec 2009) | 8 lines

          Use context from which Macro is executed, not macro context, if applicable.
          Also, ensure that the extension COULD match, not just that it won't match more.
          (closes issue ASTERISK-15021)
          Reported by: OrNix
          Patches:
          20091216__issue16113.diff.txt uploaded by tilghman (license 14)
          Tested by: OrNix
          ........

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

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

          Show
          Digium Subversion added a comment - Repository: asterisk Revision: 235422 _U trunk/ U trunk/main/pbx.c ------------------------------------------------------------------------ r235422 | tilghman | 2009-12-17 11:19:08 -0600 (Thu, 17 Dec 2009) | 15 lines Merged revisions 235421 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r235421 | tilghman | 2009-12-17 11:17:51 -0600 (Thu, 17 Dec 2009) | 8 lines Use context from which Macro is executed, not macro context, if applicable. Also, ensure that the extension COULD match, not just that it won't match more. (closes issue ASTERISK-15021 ) Reported by: OrNix Patches: 20091216__issue16113.diff.txt uploaded by tilghman (license 14) Tested by: OrNix ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=235422
          Hide
          Digium Subversion added a comment -

          Repository: asterisk
          Revision: 235423

          _U branches/1.6.0/
          U branches/1.6.0/main/pbx.c

          ------------------------------------------------------------------------
          r235423 | tilghman | 2009-12-17 11:24:29 -0600 (Thu, 17 Dec 2009) | 22 lines

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

          ................
          r235422 | tilghman | 2009-12-17 11:19:08 -0600 (Thu, 17 Dec 2009) | 15 lines

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

          ........
          r235421 | tilghman | 2009-12-17 11:17:51 -0600 (Thu, 17 Dec 2009) | 8 lines

          Use context from which Macro is executed, not macro context, if applicable.
          Also, ensure that the extension COULD match, not just that it won't match more.
          (closes issue ASTERISK-15021)
          Reported by: OrNix
          Patches:
          20091216__issue16113.diff.txt uploaded by tilghman (license 14)
          Tested by: OrNix
          ........
          ................

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

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

          Show
          Digium Subversion added a comment - Repository: asterisk Revision: 235423 _U branches/1.6.0/ U branches/1.6.0/main/pbx.c ------------------------------------------------------------------------ r235423 | tilghman | 2009-12-17 11:24:29 -0600 (Thu, 17 Dec 2009) | 22 lines Merged revisions 235422 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r235422 | tilghman | 2009-12-17 11:19:08 -0600 (Thu, 17 Dec 2009) | 15 lines Merged revisions 235421 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r235421 | tilghman | 2009-12-17 11:17:51 -0600 (Thu, 17 Dec 2009) | 8 lines Use context from which Macro is executed, not macro context, if applicable. Also, ensure that the extension COULD match, not just that it won't match more. (closes issue ASTERISK-15021 ) Reported by: OrNix Patches: 20091216__issue16113.diff.txt uploaded by tilghman (license 14) Tested by: OrNix ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=235423
          Hide
          Digium Subversion added a comment -

          Repository: asterisk
          Revision: 235424

          _U branches/1.6.1/
          U branches/1.6.1/main/pbx.c

          ------------------------------------------------------------------------
          r235424 | tilghman | 2009-12-17 11:24:38 -0600 (Thu, 17 Dec 2009) | 22 lines

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

          ................
          r235422 | tilghman | 2009-12-17 11:19:08 -0600 (Thu, 17 Dec 2009) | 15 lines

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

          ........
          r235421 | tilghman | 2009-12-17 11:17:51 -0600 (Thu, 17 Dec 2009) | 8 lines

          Use context from which Macro is executed, not macro context, if applicable.
          Also, ensure that the extension COULD match, not just that it won't match more.
          (closes issue ASTERISK-15021)
          Reported by: OrNix
          Patches:
          20091216__issue16113.diff.txt uploaded by tilghman (license 14)
          Tested by: OrNix
          ........
          ................

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

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

          Show
          Digium Subversion added a comment - Repository: asterisk Revision: 235424 _U branches/1.6.1/ U branches/1.6.1/main/pbx.c ------------------------------------------------------------------------ r235424 | tilghman | 2009-12-17 11:24:38 -0600 (Thu, 17 Dec 2009) | 22 lines Merged revisions 235422 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r235422 | tilghman | 2009-12-17 11:19:08 -0600 (Thu, 17 Dec 2009) | 15 lines Merged revisions 235421 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r235421 | tilghman | 2009-12-17 11:17:51 -0600 (Thu, 17 Dec 2009) | 8 lines Use context from which Macro is executed, not macro context, if applicable. Also, ensure that the extension COULD match, not just that it won't match more. (closes issue ASTERISK-15021 ) Reported by: OrNix Patches: 20091216__issue16113.diff.txt uploaded by tilghman (license 14) Tested by: OrNix ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=235424
          Hide
          Digium Subversion added a comment -

          Repository: asterisk
          Revision: 235425

          _U branches/1.6.2/
          U branches/1.6.2/main/pbx.c

          ------------------------------------------------------------------------
          r235425 | tilghman | 2009-12-17 11:24:50 -0600 (Thu, 17 Dec 2009) | 22 lines

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

          ................
          r235422 | tilghman | 2009-12-17 11:19:08 -0600 (Thu, 17 Dec 2009) | 15 lines

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

          ........
          r235421 | tilghman | 2009-12-17 11:17:51 -0600 (Thu, 17 Dec 2009) | 8 lines

          Use context from which Macro is executed, not macro context, if applicable.
          Also, ensure that the extension COULD match, not just that it won't match more.
          (closes issue ASTERISK-15021)
          Reported by: OrNix
          Patches:
          20091216__issue16113.diff.txt uploaded by tilghman (license 14)
          Tested by: OrNix
          ........
          ................

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

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

          Show
          Digium Subversion added a comment - Repository: asterisk Revision: 235425 _U branches/1.6.2/ U branches/1.6.2/main/pbx.c ------------------------------------------------------------------------ r235425 | tilghman | 2009-12-17 11:24:50 -0600 (Thu, 17 Dec 2009) | 22 lines Merged revisions 235422 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r235422 | tilghman | 2009-12-17 11:19:08 -0600 (Thu, 17 Dec 2009) | 15 lines Merged revisions 235421 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r235421 | tilghman | 2009-12-17 11:17:51 -0600 (Thu, 17 Dec 2009) | 8 lines Use context from which Macro is executed, not macro context, if applicable. Also, ensure that the extension COULD match, not just that it won't match more. (closes issue ASTERISK-15021 ) Reported by: OrNix Patches: 20091216__issue16113.diff.txt uploaded by tilghman (license 14) Tested by: OrNix ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=235425

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: