Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Labels:
      None
    • Mantis ID:
      13626
    • Regression:
      No

      Description

      If executing Dial with timeout 0, it places a call and then CANCEL is sent immediately before receiving TRYING message.

      Related to ASTERISK-12828
      Asterisk 1.6.0

      1. 13626.patch
        0.9 kB
        Mark Michelson
      2. dial_0_timeout.log
        19 kB

        Activity

        Hide
        Leif Madsen added a comment -

        It is possible this may make it into 1.6.1 as this appears to be a bug. It will be up to the developer who merges this to determine where to merge, but I suspect this will be in 1.6.1.

        Show
        Leif Madsen added a comment - It is possible this may make it into 1.6.1 as this appears to be a bug. It will be up to the developer who merges this to determine where to merge, but I suspect this will be in 1.6.1.
        Hide
        Leif Madsen added a comment -

        I said I wouldn't assign anymore to putnopvut, but since this one is tested and seems to fix the issue, I'm hoping he can take this "one last issue(tm)" and get it committed

        Show
        Leif Madsen added a comment - I said I wouldn't assign anymore to putnopvut, but since this one is tested and seems to fix the issue, I'm hoping he can take this "one last issue(tm)" and get it committed
        Hide
        Atis Lezdins added a comment -

        I think that it would be good that either this issue or ASTERISK-12828 is fixed in 1.6.0.

        Currently if you would try Dial with timeout 0, our Linksys PAP2 would just ring and ring without ever stopping. Picking up hook would just give you dialtone.

        Show
        Atis Lezdins added a comment - I think that it would be good that either this issue or ASTERISK-12828 is fixed in 1.6.0. Currently if you would try Dial with timeout 0, our Linksys PAP2 would just ring and ring without ever stopping. Picking up hook would just give you dialtone.
        Hide
        Mark Michelson added a comment -

        Ah, sorry. I must have missed the note where you mentioned that the patch fixed your problem, otherwise I would have merged this back then. It's going in now, though , and I'll go ahead and put it in 1.6.0, too.

        Show
        Mark Michelson added a comment - Ah, sorry. I must have missed the note where you mentioned that the patch fixed your problem, otherwise I would have merged this back then. It's going in now, though , and I'll go ahead and put it in 1.6.0, too.
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 155467

        U trunk/channels/chan_sip.c

        ------------------------------------------------------------------------
        r155467 | mmichelson | 2008-11-07 17:41:39 -0600 (Fri, 07 Nov 2008) | 12 lines

        Set the invite state to INV_CANCELLED in a place that
        makes more sense. Where it was set before, it was impossible
        to actually delay sending a CANCEL if we had not yet received
        a provisional response to an INVITE.

        (closes issue ASTERISK-12829)
        Reported by: atis
        Patches:
        13626.patch uploaded by putnopvut (license 60)
        Tested by: atis

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

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

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 155467 U trunk/channels/chan_sip.c ------------------------------------------------------------------------ r155467 | mmichelson | 2008-11-07 17:41:39 -0600 (Fri, 07 Nov 2008) | 12 lines Set the invite state to INV_CANCELLED in a place that makes more sense. Where it was set before, it was impossible to actually delay sending a CANCEL if we had not yet received a provisional response to an INVITE. (closes issue ASTERISK-12829 ) Reported by: atis Patches: 13626.patch uploaded by putnopvut (license 60) Tested by: atis ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=155467

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development