WILC_MALLOC(size) is wrapping a call to kmalloc(size, GFP_ATOMIC) with a
check for 'size > 0', which kmalloc handles anyway
The semantic patch that makes this change is as follows:
@@
expression v, s;
type t;
identifier i;
@@
(
- v = WILC_MALLOC(s);
+ v = kmalloc(s, GFP_ATOMIC);
|
- t i = WILC_MALLOC(s);
+ t i = kmalloc(s, GFP_ATOMIC);
)
Signed-off-by: Mike Rapoport <mike.rapop...@gmail.com>
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 56 +++++++++++++++--------
1 file changed, 36 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
index cf76a33..ed60e2a 100644
--- a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
+++ b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
@@ -341,8 +341,8 @@ void add_network_to_shadow(tstrNetworkInfo
*pstrNetworkInfo, void *pUserVoid, vo
astrLastScannedNtwrksShadow[ap_index].u64Tsf = pstrNetworkInfo->u64Tsf;
if (ap_found != -1)
kfree(astrLastScannedNtwrksShadow[ap_index].pu8IEs);
- astrLastScannedNtwrksShadow[ap_index].pu8IEs =
- WILC_MALLOC(pstrNetworkInfo->u16IEsLen); /* will be
deallocated by the WILC_WFI_CfgScan() function */
+ astrLastScannedNtwrksShadow[ap_index].pu8IEs =
kmalloc(pstrNetworkInfo->u16IEsLen,
+ GFP_ATOMIC);
/* will be deallocated by the WILC_WFI_CfgScan() function */
memcpy(astrLastScannedNtwrksShadow[ap_index].pu8IEs,
pstrNetworkInfo->pu8IEs, pstrNetworkInfo->u16IEsLen);
@@ -763,7 +763,8 @@ static int WILC_WFI_CfgScan(struct wiphy *wiphy, struct
cfg80211_scan_request *r
if (request->n_ssids >= 1) {
- strHiddenNetwork.pstrHiddenNetworkInfo =
WILC_MALLOC(request->n_ssids * sizeof(tstrHiddenNetwork));
+ strHiddenNetwork.pstrHiddenNetworkInfo =
kmalloc(request->n_ssids * sizeof(tstrHiddenNetwork),
+
GFP_ATOMIC);
strHiddenNetwork.u8ssidnum = request->n_ssids;
@@ -771,7 +772,8 @@ static int WILC_WFI_CfgScan(struct wiphy *wiphy, struct
cfg80211_scan_request *r
for (i = 0; i < request->n_ssids; i++) {
if (request->ssids[i].ssid != NULL &&
request->ssids[i].ssid_len != 0) {
-
strHiddenNetwork.pstrHiddenNetworkInfo[i].pu8ssid =
WILC_MALLOC(request->ssids[i].ssid_len);
+
strHiddenNetwork.pstrHiddenNetworkInfo[i].pu8ssid =
kmalloc(request->ssids[i].ssid_len,
+
GFP_ATOMIC);
memcpy(strHiddenNetwork.pstrHiddenNetworkInfo[i].pu8ssid,
request->ssids[i].ssid, request->ssids[i].ssid_len);
strHiddenNetwork.pstrHiddenNetworkInfo[i].u8ssidlen =
request->ssids[i].ssid_len;
} else {
@@ -927,7 +929,8 @@ static int WILC_WFI_CfgConnect(struct wiphy *wiphy, struct
net_device *dev,
/*BugID_5137*/
g_key_wep_params.key_len = sme->key_len;
- g_key_wep_params.key = WILC_MALLOC(sme->key_len);
+ g_key_wep_params.key = kmalloc(sme->key_len,
+ GFP_ATOMIC);
memcpy(g_key_wep_params.key, sme->key, sme->key_len);
g_key_wep_params.key_idx = sme->key_idx;
g_wep_keys_saved = true;
@@ -945,7 +948,8 @@ static int WILC_WFI_CfgConnect(struct wiphy *wiphy, struct
net_device *dev,
/*BugID_5137*/
g_key_wep_params.key_len = sme->key_len;
- g_key_wep_params.key = WILC_MALLOC(sme->key_len);
+ g_key_wep_params.key = kmalloc(sme->key_len,
+ GFP_ATOMIC);
memcpy(g_key_wep_params.key, sme->key, sme->key_len);
g_key_wep_params.key_idx = sme->key_idx;
g_wep_keys_saved = true;
@@ -1199,13 +1203,15 @@ static int WILC_WFI_add_key(struct wiphy *wiphy, struct
net_device *netdev, u8 k
if (priv->wdev->iftype == NL80211_IFTYPE_AP ||
priv->wdev->iftype == NL80211_IFTYPE_P2P_GO) {
if (priv->wilc_gtk[key_index] == NULL) {
- priv->wilc_gtk[key_index] =
WILC_MALLOC(sizeof(struct wilc_wfi_key));
+ priv->wilc_gtk[key_index] =
kmalloc(sizeof(struct wilc_wfi_key),
+ GFP_ATOMIC);
priv->wilc_gtk[key_index]->key = NULL;
priv->wilc_gtk[key_index]->seq = NULL;
}
if (priv->wilc_ptk[key_index] == NULL) {
- priv->wilc_ptk[key_index] =
WILC_MALLOC(sizeof(struct wilc_wfi_key));
+ priv->wilc_ptk[key_index] =
kmalloc(sizeof(struct wilc_wfi_key),
+ GFP_ATOMIC);
priv->wilc_ptk[key_index]->key = NULL;
priv->wilc_ptk[key_index]->seq = NULL;
}
@@ -1230,7 +1236,8 @@ static int WILC_WFI_add_key(struct wiphy *wiphy, struct
net_device *netdev, u8 k
if (priv->wilc_gtk[key_index]->key)
kfree(priv->wilc_gtk[key_index]->key);
- priv->wilc_gtk[key_index]->key =
WILC_MALLOC(params->key_len);
+ priv->wilc_gtk[key_index]->key =
kmalloc(params->key_len,
+
GFP_ATOMIC);
memcpy(priv->wilc_gtk[key_index]->key,
params->key, params->key_len);
/* if there has been previous allocation for
the same index through its seq, free that memory and allocate again*/
@@ -1238,7 +1245,8 @@ static int WILC_WFI_add_key(struct wiphy *wiphy, struct
net_device *netdev, u8 k
kfree(priv->wilc_gtk[key_index]->seq);
if ((params->seq_len) > 0) {
- priv->wilc_gtk[key_index]->seq =
WILC_MALLOC(params->seq_len);
+ priv->wilc_gtk[key_index]->seq =
kmalloc(params->seq_len,
+
GFP_ATOMIC);
memcpy(priv->wilc_gtk[key_index]->seq,
params->seq, params->seq_len);
}
@@ -1276,13 +1284,15 @@ static int WILC_WFI_add_key(struct wiphy *wiphy, struct
net_device *netdev, u8 k
if (priv->wilc_ptk[key_index]->key)
kfree(priv->wilc_ptk[key_index]->key);
- priv->wilc_ptk[key_index]->key =
WILC_MALLOC(params->key_len);
+ priv->wilc_ptk[key_index]->key =
kmalloc(params->key_len,
+
GFP_ATOMIC);
if (priv->wilc_ptk[key_index]->seq)
kfree(priv->wilc_ptk[key_index]->seq);
if ((params->seq_len) > 0)
- priv->wilc_ptk[key_index]->seq =
WILC_MALLOC(params->seq_len);
+ priv->wilc_ptk[key_index]->seq =
kmalloc(params->seq_len,
+
GFP_ATOMIC);
if (INFO) {
for (i = 0; i < params->key_len; i++)
@@ -1326,15 +1336,18 @@ static int WILC_WFI_add_key(struct wiphy *wiphy, struct
net_device *netdev, u8 k
if (!mac_addr) {
g_add_gtk_key_params.mac_addr =
NULL;
} else {
- g_add_gtk_key_params.mac_addr =
WILC_MALLOC(ETH_ALEN);
+ g_add_gtk_key_params.mac_addr =
kmalloc(ETH_ALEN,
+
GFP_ATOMIC);
memcpy(g_add_gtk_key_params.mac_addr, mac_addr, ETH_ALEN);
}
g_key_gtk_params.key_len =
params->key_len;
g_key_gtk_params.seq_len =
params->seq_len;
- g_key_gtk_params.key =
WILC_MALLOC(params->key_len);
+ g_key_gtk_params.key =
kmalloc(params->key_len,
+
GFP_ATOMIC);
memcpy(g_key_gtk_params.key,
params->key, params->key_len);
if (params->seq_len > 0) {
- g_key_gtk_params.seq =
WILC_MALLOC(params->seq_len);
+ g_key_gtk_params.seq =
kmalloc(params->seq_len,
+
GFP_ATOMIC);
memcpy(g_key_gtk_params.seq,
params->seq, params->seq_len);
}
g_key_gtk_params.cipher =
params->cipher;
@@ -1363,15 +1376,18 @@ static int WILC_WFI_add_key(struct wiphy *wiphy, struct
net_device *netdev, u8 k
if (!mac_addr) {
g_add_ptk_key_params.mac_addr =
NULL;
} else {
- g_add_ptk_key_params.mac_addr =
WILC_MALLOC(ETH_ALEN);
+ g_add_ptk_key_params.mac_addr =
kmalloc(ETH_ALEN,
+
GFP_ATOMIC);
memcpy(g_add_ptk_key_params.mac_addr, mac_addr, ETH_ALEN);
}
g_key_ptk_params.key_len =
params->key_len;
g_key_ptk_params.seq_len =
params->seq_len;
- g_key_ptk_params.key =
WILC_MALLOC(params->key_len);
+ g_key_ptk_params.key =
kmalloc(params->key_len,
+
GFP_ATOMIC);
memcpy(g_key_ptk_params.key,
params->key, params->key_len);
if (params->seq_len > 0) {
- g_key_ptk_params.seq =
WILC_MALLOC(params->seq_len);
+ g_key_ptk_params.seq =
kmalloc(params->seq_len,
+
GFP_ATOMIC);
memcpy(g_key_ptk_params.seq,
params->seq, params->seq_len);
}
g_key_ptk_params.cipher =
params->cipher;
@@ -2525,12 +2541,12 @@ int WILC_WFI_mgmt_tx(struct wiphy *wiphy,
if (ieee80211_is_mgmt(mgmt->frame_control)) {
/*mgmt frame allocation*/
- mgmt_tx = WILC_MALLOC(sizeof(struct p2p_mgmt_data));
+ mgmt_tx = kmalloc(sizeof(struct p2p_mgmt_data), GFP_ATOMIC);
if (mgmt_tx == NULL) {
PRINT_ER("Failed to allocate memory for mgmt_tx
structure\n");
return WILC_FAIL;
}
- mgmt_tx->buff = WILC_MALLOC(buf_len);
+ mgmt_tx->buff = kmalloc(buf_len, GFP_ATOMIC);
if (mgmt_tx->buff == NULL) {
PRINT_ER("Failed to allocate memory for mgmt_tx
buff\n");
kfree(mgmt_tx);
--
2.1.0
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
