[asterisk-bugs] [Asterisk 0014385]: Unhold fails if first SDP on OK, particularly Cisco CCM 6

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14385 
====================================================================== 
Reported By:                davidw
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14385
Category:                   Channels/chan_sip/Interoperability
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.0.1 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-02-02 06:39 CST
Last Modified:              2009-02-16 05:35 CST
====================================================================== 
Summary:                    Unhold fails if first SDP on OK, particularly Cisco
CCM 6
Description: 
Cisco CCM 6 appears never to send SDP on the INVITE, so Asterisk
effectively makes the offer, even when the Cisco is re-inviting because a
CCM hosted phone has gone on hold.  Asterisk sets up its SDP from flags[1]
in the channel private data structure, which means that, if the channel was
on hold, it will offer a=inactive (observed, although a=recvonly also seems
possible).  Even if the CCM intended to unhold, the only possible response
to an a=inactive offer, is a=inactive.  Asterisk should only actually use
saved state if it is making the response.  For an offer, as it looks like
it can never locally hold, it should offer a=sendrecv.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0014448 [patch] chan_sip fails to remove hold w...
====================================================================== 

---------------------------------------------------------------------- 
 (0100173) davidw (reporter) - 2009-02-16 05:35
 http://bugs.digium.com/view.php?id=14385#c100173 
---------------------------------------------------------------------- 
As far as I can tell, the patch on which
http://bugs.digium.com/view.php?id=14448 depends never made it
into trunk, rather than being subsequently taken out.  That patch is edit
65041, and looks like it didn't go through the issues system:

Modified Fri May 18 12:58:39 2007 UTC (21 months ago) by oej 
File length: 641517 byte(s) 

- Adding support for putting calls OFF hold with a re-invite with blank
SDP. This was a bug found while doing tests at SIPit in Antwerp.
- In order to not duplicate code, I restructured some of the code for
putting calls on/off hold.

Thanks DEA for reminding me. This fix has been asleep in the videocaps
branch until now. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-02-16 05:35 davidw         Note Added: 0100173                          
======================================================================