Merged revisions 313190 via svnmerge from
authorRichard Mudgett <rmudgett@digium.com>
Mon, 11 Apr 2011 15:47:17 +0000 (15:47 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Mon, 11 Apr 2011 15:47:17 +0000 (15:47 +0000)
commitbc907695bf295306415d488440bd5253bf00b589
tree2965fca440a825ed14fb6582ba3e5415af917efa
parent723215f4ed0a981bd0348abbc0a4b5850ef60902
Merged revisions 313190 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r313190 | rmudgett | 2011-04-11 10:40:30 -0500 (Mon, 11 Apr 2011) | 39 lines

  Merged revisions 313189 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2

  ................
    r313189 | rmudgett | 2011-04-11 10:32:53 -0500 (Mon, 11 Apr 2011) | 32 lines

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

    ........
      r313188 | rmudgett | 2011-04-11 10:27:52 -0500 (Mon, 11 Apr 2011) | 25 lines

      Stuck channel using FEATD_MF if caller hangs up at the right time.

      The cause was actually a caller hanging up just at the end of the Feature
      Group D DTMF tones that setup the call.  The reason for this is a "guard
      timer" that's implemented using ast_safe_sleep(100).  If the caller
      happens to hang up AFTER the final tone of the DTMF string but BEFORE the
      end of that ast_safe_sleep(), then ast_safe_sleep() will return non-zero.
      This causes the code to bounce to the end of ss_thread(), but it does NOT
      tear down the call properly.

      This should be a rare occurrence because the caller has to hang up at
      EXACTLY the right time.  Nonetheless, it was happening quite regularly on
      the reporter's system.  It's not easily reproducible, unless you purposely
      increase the guard-time to 2000 or more.  Once you do that, you can
      reproduce it every time by watching the DTMF debug and hanging up just as
      it ends.

      Simply add an ast_hangup() before goto quit.

      (closes issue #15671)
      Reported by: jcromes
      Patches:
            issue15671.patch uploaded by pabelanger (license 224)
      Tested by: jcromes
    ........
  ................
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@313191 65c4cc65-6c06-0410-ace0-fbb531ad65f3
channels/chan_dahdi.c
channels/sig_analog.c