[asterisk-dev] [Code Review] 3373: app_confbridge: Users with start_muted set aren't starting muted (although the first appears to)

[Jonathan Rose]

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

Review request for Asterisk Developers, Matt Jordan and rmudgett.


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


Repository: Asterisk


Description
-------

Long story short, the mute flag is raised after the conference user is joined to the conference bridge. During the join, the user's mute flag (which is still unraised at this point) is checked to determine initial mute status. The first user appears to work because a second update occurs on it when a second user joins (this can be broken by having the second user join while the first user is still in the process of joining).

This patch fixes all of that simply by setting the muted flag prior to joining.


Diffs
-----

  /branches/11/apps/app_confbridge.c 410862 

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


Testing
-------

Tested joining with 1, 2, and 3 participants. Checked practical mute status (IE who hears what) and stated mute status in each condition from 1 participant to 3 participants.

I'm also currently writing a testsuite test.


Thanks,

Jonathan Rose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140318/53a5579c/attachment.html>