From 034f56f31e60b3391b5341a4951ef8342aa6307f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=A9bastien=20Deronne?= Date: Sun, 12 Jan 2020 12:12:54 +0100 Subject: [PATCH] wifi: Fix inverted logic to call capabilities changed callback only if either the channel width changed or the number of spatial streams changed --- src/wifi/model/wifi-phy.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/wifi/model/wifi-phy.cc b/src/wifi/model/wifi-phy.cc index 82a6ea6ef..a003df923 100644 --- a/src/wifi/model/wifi-phy.cc +++ b/src/wifi/model/wifi-phy.cc @@ -1344,7 +1344,7 @@ WifiPhy::SetChannelWidth (uint16_t channelwidth) { NS_LOG_FUNCTION (this << channelwidth); NS_ASSERT_MSG (channelwidth == 5 || channelwidth == 10 || channelwidth == 20 || channelwidth == 22 || channelwidth == 40 || channelwidth == 80 || channelwidth == 160, "wrong channel width value"); - bool changed = (m_channelWidth == channelwidth); + bool changed = (m_channelWidth != channelwidth); m_channelWidth = channelwidth; AddSupportedChannelWidth (channelwidth); if (changed && !m_capabilitiesChangedCallback.IsNull ()) @@ -1377,7 +1377,7 @@ void WifiPhy::SetMaxSupportedTxSpatialStreams (uint8_t streams) { NS_ASSERT (streams <= GetNumberOfAntennas ()); - bool changed = (m_txSpatialStreams == streams); + bool changed = (m_txSpatialStreams != streams); m_txSpatialStreams = streams; ConfigureHtDeviceMcsSet (); if (changed && !m_capabilitiesChangedCallback.IsNull ()) @@ -1396,7 +1396,7 @@ void WifiPhy::SetMaxSupportedRxSpatialStreams (uint8_t streams) { NS_ASSERT (streams <= GetNumberOfAntennas ()); - bool changed = (m_rxSpatialStreams == streams); + bool changed = (m_rxSpatialStreams != streams); m_rxSpatialStreams = streams; if (changed && !m_capabilitiesChangedCallback.IsNull ()) {