From 2c2f23c996acdf18ef0f3a7fb28e0b39ed12f04a Mon Sep 17 00:00:00 2001 From: Itay Elenzweig Date: Sun, 12 Apr 2020 17:57:11 +0300 Subject: [PATCH] son_master_thread: remove unneeded use of set_hostap_supported_channels In handle_intel_slave_join, there is a call to set_hostap_supported_channels, the function is also used as part of add_hostap_supported_operating_class which is called by autoconfig_wsc_parse_radio_caps, which runs in the handle_intel_slave_join making the first use of set_hostap_supported_channels redundant. Remove set_hostap_supported_channels from handle_intel_slave_join Signed-off-by: Itay Elenzweig --- controller/src/beerocks/master/son_master_thread.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/controller/src/beerocks/master/son_master_thread.cpp b/controller/src/beerocks/master/son_master_thread.cpp index 98074a9a67..d6061f6364 100644 --- a/controller/src/beerocks/master/son_master_thread.cpp +++ b/controller/src/beerocks/master/son_master_thread.cpp @@ -2101,8 +2101,6 @@ bool master_thread::handle_intel_slave_join( database.set_node_ipv4(radio_mac, bridge_ipv4); database.set_node_manufacturer(radio_mac, "Intel"); - database.set_hostap_supported_channels(radio_mac, notification->hostap().supported_channels, - message::SUPPORTED_CHANNELS_LENGTH); if (database.get_node_5ghz_support(radio_mac)) { if (notification->low_pass_filter_on()) { @@ -2272,7 +2270,7 @@ bool master_thread::autoconfig_wsc_parse_radio_caps( non_operable_channels.push_back(*channel); } ss << " }" << std::endl; - // LOG(DEBUG) << ss.str(); + LOG(DEBUG) << ss.str(); // store operating class in the DB for this hostap database.add_hostap_supported_operating_class( radio_mac, operating_class, maximum_transmit_power_dbm, non_operable_channels);