[Home]

Summary:ASTERISK-03705: [patch] INVITE to Polycom 500 broken after handle_request patch
Reporter:kuj (kuj)Labels:
Date Opened:2005-03-18 18:14:00.000-0600Date Closed:2008-01-15 15:27:53.000-0600
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) missingtag.txt
( 1) polyfail.txt
( 2) polysuccess676.txt
Description:After installing chan_sip.c from HEAD (1.677, based on ASTERISK-3692), SIP INVITEs to a Polycom IP 500 do not succeed anymore. Even though * reports "Call answered" and "Attempting native bridge", no audio is bridged.

****** ADDITIONAL INFORMATION ******

OK, this is a weird one: running * CVS HEAD, one extension a Polycom 500, the other a SPA2K. Any call (from the SPA, from a ZAP channel, from external SIP providers) to the Polycom cannot be "picked up". The rtp bridge does not appear to get established. This problem goes away when reverting to chan_sip 1.676 (the version before the handle_request rewrite).

However, calling from the Polycom does succeed, with bridged audio. Also, in this latest version, picking up a call on the SPA does succeed in all cases where the Polycom will not .

I've attached two sip debug traces, both where 2001 (SPA) invites 2006 (Polycom). In polyfail, it seems that even after the attempted native bridge, * and the phone won't stop trying to (re?)negotiate call parameters. polysuccess767 (using chan_sip 1.676) lists a successful call.

Note: Both phones (friends) are set to canreinvite=no
Comments:By: Olle Johansson (oej) 2005-03-19 04:08:26.000-0600

Seems like the phone does not get our ACK...
By: Olle Johansson (oej) 2005-03-19 04:21:07.000-0600

I see the problem. The TAG in To: is missing in the ACK when we fail. Ok, I'll check the source and see where it got lost.
By: Olle Johansson (oej) 2005-03-19 04:28:00.000-0600

Ok, found the bug. Sorry, my mistake. Thank you for a detailed report, it really helped me finding this!
By: kuj (kuj) 2005-03-19 11:28:44.000-0600

That was it. Thanks much!
By: Mark Spencer (markster) 2005-03-19 11:36:45.000-0600

Fixed in CVS head, thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:27:53.000-0600

Repository: asterisk
Revision: 5204

U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r5204 | markster | 2008-01-15 15:27:52 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix tag issue (bug ASTERISK-3705)

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

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