wifi: Fixed potential bug with broadcasted packets when RTS-CTS prote…

…ction is used
digideskio · Nov 7, 2016 · 7a7a87a · 7a7a87a
1 parent 3a1d92c
commit 7a7a87a
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/wifi/model/wifi-remote-station-manager.cc b/src/wifi/model/wifi-remote-station-manager.cc
@@ -897,6 +897,10 @@ WifiRemoteStationManager::NeedRts (Mac48Address address, const WifiMacHeader *he
 {
   WifiMode mode = txVector.GetMode ();
   NS_LOG_FUNCTION (this << address << *header << packet << mode);
+  if (address.IsGroup ())
+    {
+      return false;
+    }
   if (m_protectionMode == RTS_CTS
       && ((mode.GetModulationClass () == WIFI_MOD_CLASS_ERP_OFDM)
       || (mode.GetModulationClass () == WIFI_MOD_CLASS_HT)
@@ -906,10 +910,6 @@ WifiRemoteStationManager::NeedRts (Mac48Address address, const WifiMacHeader *he
       NS_LOG_DEBUG ("WifiRemoteStationManager::NeedRTS returning true to protect non-ERP stations");
       return true;
     }
-  if (address.IsGroup ())
-    {
-      return false;
-    }
   bool normally = (packet->GetSize () + header->GetSize () + WIFI_MAC_FCS_LENGTH) > GetRtsCtsThreshold ();
   return DoNeedRts (Lookup (address, header), packet, normally);
 }