From b437656c2ebebe9d473454585e72d9cbab9bf474 Mon Sep 17 00:00:00 2001 From: Chris-Savinovich Date: Wed, 18 Apr 2018 12:41:50 -0400 Subject: "confbridge show profile bridge" does not output "sfu" when video_mode is sfu Fixes a bug on the "confbridge show profile bridge" cli command that showed "video_mode=no video" when video_mode was set to "sfu" ASTERISK-27418 #close Change-Id: I481e3172c7f872664c7ac7809879d541c9f031e9 --- apps/confbridge/conf_config_parser.c | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/apps/confbridge/conf_config_parser.c b/apps/confbridge/conf_config_parser.c index c143e39e2..873831911 100644 --- a/apps/confbridge/conf_config_parser.c +++ b/apps/confbridge/conf_config_parser.c @@ -1673,8 +1673,10 @@ static char *handle_cli_confbridge_show_bridge_profile(struct ast_cli_entry *e, ast_cli(a->fd,"Registration context: %s\n", b_profile.regcontext); switch (b_profile.flags - & (BRIDGE_OPT_VIDEO_SRC_LAST_MARKED | BRIDGE_OPT_VIDEO_SRC_FIRST_MARKED - | BRIDGE_OPT_VIDEO_SRC_FOLLOW_TALKER)) { + & (BRIDGE_OPT_VIDEO_SRC_LAST_MARKED | + BRIDGE_OPT_VIDEO_SRC_FIRST_MARKED | + BRIDGE_OPT_VIDEO_SRC_FOLLOW_TALKER | + BRIDGE_OPT_VIDEO_SRC_SFU)) { case BRIDGE_OPT_VIDEO_SRC_LAST_MARKED: ast_cli(a->fd, "Video Mode: last_marked\n"); break; @@ -1684,6 +1686,9 @@ static char *handle_cli_confbridge_show_bridge_profile(struct ast_cli_entry *e, case BRIDGE_OPT_VIDEO_SRC_FOLLOW_TALKER: ast_cli(a->fd, "Video Mode: follow_talker\n"); break; + case BRIDGE_OPT_VIDEO_SRC_SFU: + ast_cli(a->fd, "Video Mode: sfu\n"); + break; case 0: ast_cli(a->fd, "Video Mode: no video\n"); break; @@ -2030,12 +2035,6 @@ static int video_mode_handler(const struct aco_option *opt, struct ast_variable | BRIDGE_OPT_VIDEO_SRC_FOLLOW_TALKER | BRIDGE_OPT_VIDEO_SRC_SFU, BRIDGE_OPT_VIDEO_SRC_FOLLOW_TALKER); - } else if (!strcasecmp(var->value, "none")) { - ast_clear_flag(b_profile, - BRIDGE_OPT_VIDEO_SRC_FIRST_MARKED - | BRIDGE_OPT_VIDEO_SRC_LAST_MARKED - | BRIDGE_OPT_VIDEO_SRC_FOLLOW_TALKER - | BRIDGE_OPT_VIDEO_SRC_SFU); } else if (!strcasecmp(var->value, "sfu")) { ast_set_flags_to(b_profile, BRIDGE_OPT_VIDEO_SRC_FIRST_MARKED @@ -2043,6 +2042,12 @@ static int video_mode_handler(const struct aco_option *opt, struct ast_variable | BRIDGE_OPT_VIDEO_SRC_FOLLOW_TALKER | BRIDGE_OPT_VIDEO_SRC_SFU, BRIDGE_OPT_VIDEO_SRC_SFU); + } else if (!strcasecmp(var->value, "none")) { + ast_clear_flag(b_profile, + BRIDGE_OPT_VIDEO_SRC_FIRST_MARKED + | BRIDGE_OPT_VIDEO_SRC_LAST_MARKED + | BRIDGE_OPT_VIDEO_SRC_FOLLOW_TALKER + | BRIDGE_OPT_VIDEO_SRC_SFU); } else { return -1; } -- cgit v1.2.3