asterisk users - May 2008 - Error after upgrading from 1.2.18 to 1.4.20

scenario is incoming calls on ZAP (TE410p  euro isdn) to SIP (or any 
other channel) and call is answered.

When I hangup on the ISDN side on the 1.2 then the SIP hangs up to 
immidiatly so everything is fine (se short pri debug below).

When I do the same on 1.4.20 then it take more than 30 seconds to 
disconnect when the ISDN calling party hangs up

I have tried to use the use the 1.4 zaptel with the 1.2 installation, 
that works perfect to so to me it looks like a libpri or chan_zap problem.
I do hope though that some has a hint to where I scre... up.

Freddi


OK trace from 1.2

< Protocol Discriminator: Q.931 (8)  len=13
< Call Ref: len= 2 (reference 7603/0x1DB3) (Originator)
< Message type: DISCONNECT (69)
< [08 02 80 90]
< Cause (len= 4) [ Ext: 1  Coding: CCITT (ITU) standard (0) 0: 0   
Location: User (0)
<                  Ext: 1  Cause: Normal Clearing (16), class = Normal 
Event (1) ]
< [1e 02 82 88]
< Progress Indicator (len= 4) [ Ext: 1  Coding: CCITT (ITU) standard (0) 
0: 0   Location: Public network serving the local user (2)
<                               Ext: 1  Progress Description: Inband 
information or appropriate pattern now available. (8) ]
-- Processing IE 8 (cs0, Cause)
-- Processing IE 30 (cs0, Progress Indicator)
    -- Channel 0/14, span 1 got hangup request
  == Spawn extension (Pstn-incoming-fwd, 77348855, 2) exited non-zero on 
'Zap/14-1'
NEW_HANGUP DEBUG: Calling q931_hangup, ourstate Disconnect Indication, 
peerstate Disconnect Request
 > Protocol Discriminator: Q.931 (8)  len=9
 > Call Ref: len= 2 (reference 7603/0x1DB3) (Terminator)
 > Message type: RELEASE (77)
 > [08 02 81 90]
 > Cause (len= 4) [ Ext: 1  Coding: CCITT (ITU) standard (0) 0: 0   
Location: Private network serving the local user (1)
 >                  Ext: 1  Cause: Normal Clearing (16), class = Normal 
Event (1) ]
    -- Hungup 'Zap/14-1'
< Protocol Discriminator: Q.931 (8)  len=5
< Call Ref: len= 2 (reference 7603/0x1DB3) (Originator)
< Message type: RELEASE COMPLETE (90)
NEW_HANGUP DEBUG: Calling q931_hangup, ourstate Null, peerstate Null
NEW_HANGUP DEBUG: Destroying the call, ourstate Null, peerstate Null
fonet2*CLI

BAD TRACE from 1.4  below:

fonet3*CLI>
< Protocol Discriminator: Q.931 (8)  len=13
< Call Ref: len= 2 (reference 7582/0x1D9E) (Originator)
< Message type: DISCONNECT (69)
< [08 02 80 90]
< Cause (len= 4) [ Ext: 1  Coding: CCITT (ITU) standard (0)  Spare: 0  
Location: User (0)
<                  Ext: 1  Cause: Normal Clearing (16), class = Normal 
Event (1) ]
< [1e 02 82 88]
< Progress Indicator (len= 4) [ Ext: 1  Coding: CCITT (ITU) standard 
(0)  0: 0  Location: Public network serving the local user (2)
<                               Ext: 1  Progress Description: Inband 
information or appropriate pattern now available. (8) ]
-- Processing IE 8 (cs0, Cause)
-- Processing IE 30 (cs0, Progress Indicator)
q931.c:3779 q931_receive: call 7582 on channel 13 enters state 12 
(Disconnect Indication)
fonet3*CLI>

~ 30 SECOND DELAY HERE.

fonet3*CLI>
fonet3*CLI>
< Protocol Discriminator: Q.931 (8)  len=9
< Call Ref: len= 2 (reference 7582/0x1D9E) (Originator)
< Message type: RELEASE (77)
< [08 02 80 90]
< Cause (len= 4) [ Ext: 1  Coding: CCITT (ITU) standard (0)  Spare: 0  
Location: User (0)
<                  Ext: 1  Cause: Normal Clearing (16), class = Normal 
Event (1) ]
-- Processing IE 8 (cs0, Cause)
q931.c:3754 q931_receive: call 7582 on channel 13 enters state 0 (Null)
    -- Channel 0/13, span 2 got hangup, cause 16
  == Spawn extension (Pstn-incoming-fwd, 77348855, 2) exited non-zero on 
'Zap/44-1'
    -- Executing [h at Pstn-incoming-fwd:1] Hangup("Zap/44-1",
"") in new stack
  == Spawn extension (Pstn-incoming-fwd, h, 1) exited non-zero on
'Zap/44-1'
NEW_HANGUP DEBUG: Calling q931_hangup, ourstate Null, peerstate Release 
Request
 > Protocol Discriminator: Q.931 (8)  len=9
 > Call Ref: len= 2 (reference 7582/0x1D9E) (Terminator)
 > Message type: RELEASE COMPLETE (90)
 > [08 02 81 90]
 > Cause (len= 4) [ Ext: 1  Coding: CCITT (ITU) standard (0)  Spare: 0  
Location: Private network serving the local user (1)
 >                  Ext: 1  Cause: Normal Clearing (16), class = Normal 
Event (1) ]
NEW_HANGUP DEBUG: Calling q931_hangup, ourstate Null, peerstate Null
NEW_HANGUP DEBUG: Destroying the call, ourstate Null, peerstate Null
    -- Hungup 'Zap/44-1'fonet3*CLI>

Hi I better answer my own post.

I went to the code and the issue is in q931.c

/* wait for a RELEASE so that sufficient time has passed
    for the inband audio to be heard */
           
      if (c->progressmask & PRI_PROG_INBAND_AVAILABLE)
                break;
              
Changing this line to a comment makes the 1.4 work exactly as 1.2 for 
this issue.
I think that this line should only be executed on 'outbound' pri calls, 
not on inbound.

Freddi