nl80211: don't clear bitrate_mask twice
authorJanusz Dziedzic <janusz.dziedzic@tieto.com>
Tue, 3 Dec 2013 08:50:44 +0000 (09:50 +0100)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 3 Dec 2013 14:19:54 +0000 (15:19 +0100)
Don't clear cfg80211_bitrate_mask twice in
nl80211_set_tx_bitrate_mask() function.

Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/nl80211.c

index 9cd8c6113e94f0e20dd516886b49f888467102ce..bdcf256e3628abfc3848cd8f17625672eed1c741 100644 (file)
@@ -7349,15 +7349,14 @@ static int nl80211_set_tx_bitrate_mask(struct sk_buff *skb,
        /* Default to all rates enabled */
        for (i = 0; i < IEEE80211_NUM_BANDS; i++) {
                sband = rdev->wiphy.bands[i];
-               mask.control[i].legacy =
-                       sband ? (1 << sband->n_bitrates) - 1 : 0;
-               if (sband)
-                       memcpy(mask.control[i].mcs,
-                              sband->ht_cap.mcs.rx_mask,
-                              sizeof(mask.control[i].mcs));
-               else
-                       memset(mask.control[i].mcs, 0,
-                              sizeof(mask.control[i].mcs));
+
+               if (!sband)
+                       continue;
+
+               mask.control[i].legacy = (1 << sband->n_bitrates) - 1;
+               memcpy(mask.control[i].mcs,
+                      sband->ht_cap.mcs.rx_mask,
+                      sizeof(mask.control[i].mcs));
        }
 
        /*