Merged revisions 193613 via svnmerge from
authorRichard Mudgett <rmudgett@digium.com>
Mon, 11 May 2009 19:11:29 +0000 (19:11 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Mon, 11 May 2009 19:11:29 +0000 (19:11 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r193613 | rmudgett | 2009-05-11 14:09:00 -0500 (Mon, 11 May 2009) | 12 lines

  Sent wrong message to clear a call we started if the other end has not responed yet.

  In the state MISDN_CALLING (i.e. SETUP was sent but no answer has arrived yet),
  it is not allowed to clear the call with RELEASE_COMPLETE.  It must be
  cleared with DISCONNECT.  A RELEASE_COMPLETE is only allowed as an answer
  to a SETUP.  (See Q.931 ch. 5.3.2, 5.3.2.a, 5.3.2.b)

  Patches:
      chan-misdn-ccstate7.patch uploaded by customer.

  JIRA ABE-1862
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@193614 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_misdn.c

index b356e4d..34402b0 100644 (file)
@@ -6835,12 +6835,11 @@ CLEAN_CH:
 
        switch (p->state) {
        case MISDN_INCOMING_SETUP:
-       case MISDN_CALLING:
                p->state = MISDN_CLEANING;
                /* This is the only place in misdn_hangup, where we
                 * can call release_chan, else it might create lot's of trouble
                 * */
-               ast_log(LOG_NOTICE, "release channel, in CALLING/INCOMING_SETUP state.. no other events happened\n");
+               ast_log(LOG_NOTICE, "release channel, in INCOMING_SETUP state.. no other events happened\n");
                release_chan(bc);
                misdn_lib_send_event(bc, EVENT_RELEASE_COMPLETE);
                break;
@@ -6863,6 +6862,7 @@ CLEAN_CH:
                }
                break;
 
+       case MISDN_CALLING:
        case MISDN_ALERTING:
        case MISDN_PROGRESS:
        case MISDN_PROCEEDING: