diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-11-15 08:24:57 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2012-11-16 14:29:07 -0500 |
commit | ccd953694bf9ce4639fa4d8bdb8a2a297315816f (patch) | |
tree | 2efda58086940b44897de5ca6c8bba6d6f012c3a | |
parent | 9c033bed936629155129a2b3e7493838fe76b8b5 (diff) |
rtlwifi: use eth_zero_addr() to assign zero address
Using eth_zero_addr() to assign zero address insetad of
memset() or an inefficient copy from a static array.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | drivers/net/wireless/rtlwifi/cam.c | 2 | ||||
-rw-r--r-- | drivers/net/wireless/rtlwifi/core.c | 5 |
2 files changed, 3 insertions, 4 deletions
diff --git a/drivers/net/wireless/rtlwifi/cam.c b/drivers/net/wireless/rtlwifi/cam.c index ca69e35e50f1..0e510f73041a 100644 --- a/drivers/net/wireless/rtlwifi/cam.c +++ b/drivers/net/wireless/rtlwifi/cam.c | |||
@@ -337,7 +337,7 @@ void rtl_cam_del_entry(struct ieee80211_hw *hw, u8 *sta_addr) | |||
337 | if (((bitmap & BIT(0)) == BIT(0)) && | 337 | if (((bitmap & BIT(0)) == BIT(0)) && |
338 | (memcmp(addr, sta_addr, ETH_ALEN) == 0)) { | 338 | (memcmp(addr, sta_addr, ETH_ALEN) == 0)) { |
339 | /* Remove from HW Security CAM */ | 339 | /* Remove from HW Security CAM */ |
340 | memset(rtlpriv->sec.hwsec_cam_sta_addr[i], 0, ETH_ALEN); | 340 | eth_zero_addr(rtlpriv->sec.hwsec_cam_sta_addr[i]); |
341 | rtlpriv->sec.hwsec_cam_bitmap &= ~(BIT(0) << i); | 341 | rtlpriv->sec.hwsec_cam_bitmap &= ~(BIT(0) << i); |
342 | RT_TRACE(rtlpriv, COMP_SEC, DBG_LOUD, | 342 | RT_TRACE(rtlpriv, COMP_SEC, DBG_LOUD, |
343 | "del CAM entry %d\n", i); | 343 | "del CAM entry %d\n", i); |
diff --git a/drivers/net/wireless/rtlwifi/core.c b/drivers/net/wireless/rtlwifi/core.c index a7c0e52869ba..be33aa14c8af 100644 --- a/drivers/net/wireless/rtlwifi/core.c +++ b/drivers/net/wireless/rtlwifi/core.c | |||
@@ -962,7 +962,6 @@ static int rtl_op_set_key(struct ieee80211_hw *hw, enum set_key_cmd cmd, | |||
962 | int err = 0; | 962 | int err = 0; |
963 | u8 mac_addr[ETH_ALEN]; | 963 | u8 mac_addr[ETH_ALEN]; |
964 | u8 bcast_addr[ETH_ALEN] = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }; | 964 | u8 bcast_addr[ETH_ALEN] = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }; |
965 | u8 zero_addr[ETH_ALEN] = { 0 }; | ||
966 | 965 | ||
967 | if (rtlpriv->cfg->mod_params->sw_crypto || rtlpriv->sec.use_sw_sec) { | 966 | if (rtlpriv->cfg->mod_params->sw_crypto || rtlpriv->sec.use_sw_sec) { |
968 | RT_TRACE(rtlpriv, COMP_ERR, DBG_WARNING, | 967 | RT_TRACE(rtlpriv, COMP_ERR, DBG_WARNING, |
@@ -1057,7 +1056,7 @@ static int rtl_op_set_key(struct ieee80211_hw *hw, enum set_key_cmd cmd, | |||
1057 | memcpy(rtlpriv->sec.key_buf[key_idx], | 1056 | memcpy(rtlpriv->sec.key_buf[key_idx], |
1058 | key->key, key->keylen); | 1057 | key->key, key->keylen); |
1059 | rtlpriv->sec.key_len[key_idx] = key->keylen; | 1058 | rtlpriv->sec.key_len[key_idx] = key->keylen; |
1060 | memcpy(mac_addr, zero_addr, ETH_ALEN); | 1059 | eth_zero_addr(mac_addr); |
1061 | } else if (group_key) { /* group key */ | 1060 | } else if (group_key) { /* group key */ |
1062 | RT_TRACE(rtlpriv, COMP_SEC, DBG_DMESG, | 1061 | RT_TRACE(rtlpriv, COMP_SEC, DBG_DMESG, |
1063 | "set group key\n"); | 1062 | "set group key\n"); |
@@ -1108,7 +1107,7 @@ static int rtl_op_set_key(struct ieee80211_hw *hw, enum set_key_cmd cmd, | |||
1108 | } | 1107 | } |
1109 | memset(rtlpriv->sec.key_buf[key_idx], 0, key->keylen); | 1108 | memset(rtlpriv->sec.key_buf[key_idx], 0, key->keylen); |
1110 | rtlpriv->sec.key_len[key_idx] = 0; | 1109 | rtlpriv->sec.key_len[key_idx] = 0; |
1111 | memcpy(mac_addr, zero_addr, ETH_ALEN); | 1110 | eth_zero_addr(mac_addr); |
1112 | /* | 1111 | /* |
1113 | *mac80211 will delete entrys one by one, | 1112 | *mac80211 will delete entrys one by one, |
1114 | *so don't use rtl_cam_reset_all_entry | 1113 | *so don't use rtl_cam_reset_all_entry |