----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3791/ -----------------------------------------------------------
(Updated July 15, 2014, 2:42 p.m.) Status ------ This change has been marked as submitted. Review request for Asterisk Developers. Changes ------- Committed in revision 418710 Repository: Asterisk Description ------- This fixes three bugs caught by the direct media tests in the Test Suite: (1) When adding codecs to an outgoing SDP, we failed to add the added format to the alreadysent cap structure for peer codecs. This could result in duplicate codecs getting added to the SDP. (2) Another path that associates a dialog pvt structure with a peer failed to remove the capabilities on the dialog pvt before copying over the peer's. This resulted in the 'general' section codecs getting sent in a 200 OK to an inbound INVITE, along with the peer's. (3) Direct media checks needs to make sure that things have actually changed in capabilities between checks; otherwise it will spam the far side with re-INVITE requests. Diffs ----- ./team/group/media_formats-reviewed-trunk/channels/chan_sip.c 418631 Diff: https://reviewboard.asterisk.org/r/3791/diff/ Testing ------- The sip_hold_direct_media test now passes Thanks, Matt Jordan
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
