aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>2012-11-15 08:24:57 -0500
committerJohn W. Linville <linville@tuxdriver.com>2012-11-16 14:29:07 -0500
commitccd953694bf9ce4639fa4d8bdb8a2a297315816f (patch)
tree2efda58086940b44897de5ca6c8bba6d6f012c3a
parent9c033bed936629155129a2b3e7493838fe76b8b5 (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.c2
-rw-r--r--drivers/net/wireless/rtlwifi/core.c5
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