[Asterisk-code-review] bridge_softmix: Suppress error on topology change failure (asterisk[16])

George Joseph asteriskteam at digium.com
Wed Sep 8 07:55:06 CDT 2021


George Joseph has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/16417 )


Change subject: bridge_softmix: Suppress error on topology change failure
......................................................................

bridge_softmix: Suppress error on topology change failure

There are conditions under which a failure to change topology
is expected so there's no need to print an ERROR message.

ASTERISK-29618
Reported by: Alexander

Change-Id: Idc168b8588e018bf3a23769f08c4ad646086d481
---
M bridges/bridge_softmix.c
1 file changed, 2 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/17/16417/1

diff --git a/bridges/bridge_softmix.c b/bridges/bridge_softmix.c
index d51a55f..c7b1d6c 100644
--- a/bridges/bridge_softmix.c
+++ b/bridges/bridge_softmix.c
@@ -666,7 +666,8 @@
 	ast_trace(-1, "%s: Requesting topology change.\n", ast_channel_name(joiner->chan));
 	res = ast_channel_request_stream_topology_change(joiner->chan, sc->topology, NULL);
 	if (res) {
-		SCOPE_EXIT_LOG_RTN(LOG_ERROR, "%s: Couldn't request topology change\n", ast_channel_name(joiner->chan));
+		/* There are conditions under which this is expected so no need to log an ERROR. */
+		SCOPE_EXIT_RTN("%s: Couldn't request topology change\n", ast_channel_name(joiner->chan));
 	}
 
 	AST_LIST_TRAVERSE(participants, participant, entry) {

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/16417
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: Idc168b8588e018bf3a23769f08c4ad646086d481
Gerrit-Change-Number: 16417
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210908/56dc1101/attachment.html>


More information about the asterisk-code-review mailing list