-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4429/
-----------------------------------------------------------

(Updated Feb. 24, 2015, 12:23 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 432174


Bugs: ASTERISK-24797
    https://issues.asterisk.org/jira/browse/ASTERISK-24797


Repository: Asterisk


Description
-------

When more than one call using the same codec type enters into a softmix bridge 
and no audio is present for a channel the bridge optimizes the out frame by 
using the same one for all channels with the same codec type. Unfortunately, 
when that number (channels with same codec type) dropped to <= 1 the codec was 
not dereferenced. At least not until all parties left the bridge. Thus in the 
case of G.729 the license was not released. This patch ensures that the codec 
is dereferenced immediately when the optimization no longer applies.


Diffs
-----

  branches/11/bridges/bridge_softmix.c 431876 

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


Testing
-------

Created a 3 party confbridge. Two channels were using g.729 and the other using 
ulaw. Hung up both the channels using g.729 and noted that the confbridge was 
still holding a reference to a g.729 license. After applying the patch noted 
that it no longer held onto the license.


Thanks,

Kevin Harwell

-- 
_____________________________________________________________________
-- 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

Reply via email to