[asterisk-dev] [Code Review] 3472: res_pjsip_sdp_rtp: Fix issue with unholding when it shouldn't.

Joshua Colp reviewboard at asterisk.org
Wed Apr 30 07:38:11 CDT 2014

This is an automatically generated e-mail. To reply, visit:

(Updated April 30, 2014, 7:38 a.m.)


This change has been marked as submitted.

Review request for Asterisk Developers.


Committed in revision 413119

Bugs: ASTERISK-23558

Repository: Asterisk


The code for queueing hold and unhold currently favors unholding over holding. This means that if an SDP is received with host information but with a stream state of sendonly the code will incorrectly stop musiconhold if it has been started.

The attached change fixes this by favoring hold instead of unhold. If an SDP is received with OR sendonly and hold has not been started then it is started. If it has already been started no change occurs. Otherwise if musiconhold has been started it is stopped.


  /branches/12/res/res_pjsip_sdp_rtp.c 412900 

Diff: https://reviewboard.asterisk.org/r/3472/diff/


Ran test before fixing and confirmed broken. Ran test after fixing and confirmed fixed.


Joshua Colp

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140430/6ff3ee9f/attachment.html>

More information about the asterisk-dev mailing list