[Asterisk-code-review] app confbridge: Fix bug - users with startmuted set don't st... (asterisk[certified/11.6])

Richard Mudgett asteriskteam at digium.com
Wed Jan 27 17:24:26 CST 2016


Richard Mudgett has uploaded a new change for review.

  https://gerrit.asterisk.org/2119

Change subject: app_confbridge: Fix bug - users with startmuted set don't start muted
......................................................................

app_confbridge: Fix bug - users with startmuted set don't start muted

(closes issue ASTERISK-23461)
Reported by: Chico Manobela
Review: https://reviewboard.asterisk.org/r/3373/

Cherry-picked to support ASTERISK-20987

Change-Id: I69f41779d2ae39b69ab7e7bd93d60eaccfbe3eda
---
M apps/app_confbridge.c
1 file changed, 13 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/19/2119/1

diff --git a/apps/app_confbridge.c b/apps/app_confbridge.c
index ef27a2e..f8b1325 100644
--- a/apps/app_confbridge.c
+++ b/apps/app_confbridge.c
@@ -984,6 +984,13 @@
 		ast_channel_name(user->chan), mute_effective ? "muted" : "unmuted",
 		mute_user, mute_system);
 	user->features.mute = mute_effective;
+	ast_test_suite_event_notify("CONF_MUTE_UPDATE",
+		"Mode: %s\r\n"
+		"Conference: %s\r\n"
+		"Channel: %s",
+		mute_effective ? "muted" : "unmuted",
+		user->b_profile.name,
+		ast_channel_name(user->chan));
 }
 
 void conf_moh_stop(struct conference_bridge_user *user)
@@ -1664,6 +1671,12 @@
 			conf_name);
 	}
 
+	/* If the caller should be joined already muted, set the flag before we join. */
+	if (ast_test_flag(&conference_bridge_user.u_profile, USER_OPT_STARTMUTED)) {
+		/* Set user level mute request. */
+		conference_bridge_user.muted = 1;
+	}
+
 	/* Look for a conference bridge matching the provided name */
 	if (!(conference_bridge = join_conference_bridge(args.conf_name, &conference_bridge_user))) {
 		res = -1; /* invalid PIN */
@@ -1673,12 +1686,6 @@
 	/* Keep a copy of volume adjustments so we can restore them later if need be */
 	volume_adjustments[0] = ast_audiohook_volume_get(chan, AST_AUDIOHOOK_DIRECTION_READ);
 	volume_adjustments[1] = ast_audiohook_volume_get(chan, AST_AUDIOHOOK_DIRECTION_WRITE);
-
-	/* If the caller should be joined already muted, make it so */
-	if (ast_test_flag(&conference_bridge_user.u_profile, USER_OPT_STARTMUTED)) {
-		/* Set user level mute request. */
-		conference_bridge_user.muted = 1;
-	}
 
 	if (ast_test_flag(&conference_bridge_user.u_profile, USER_OPT_DROP_SILENCE)) {
 		conference_bridge_user.tech_args.drop_silence = 1;

-- 
To view, visit https://gerrit.asterisk.org/2119
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I69f41779d2ae39b69ab7e7bd93d60eaccfbe3eda
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: certified/11.6
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Jonathan Rose <jrose at digium.com>



More information about the asterisk-code-review mailing list