staging: rtl8723bs: core: Remove typecast in call to kfree

Remove typecast in the call to kfree as it is not needed.
Issue found using the below coccinelle script with options -I and
--recursive-includes,

@@
type t1;
expression *e;
@@

-kfree((t1 *)e);
+kfree(e);

Signed-off-by: Nachammai Karuppiah <nachukannan@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Link: https://lore.kernel.org/r/1570565858-91737-1-git-send-email-nachukannan@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c b/drivers/staging/rtl8723bs/core/rtw_ap.c
index 6d18d23acdc030038ebbb5605b3a0c44684b634f..a1f501655392d7d46277c17fbb32fd6a32ce3a92 100644 (file)
--- a/drivers/staging/rtl8723bs/core/rtw_ap.c
+++ b/drivers/staging/rtl8723bs/core/rtw_ap.c
@@ -1453,7 +1453,7 @@ u8 rtw_ap_set_pairwise_key(struct adapter *padapter, struct sta_info *psta)
 
        psetstakey_para = rtw_zmalloc(sizeof(struct set_stakey_parm));
        if (psetstakey_para == NULL) {
-               kfree((u8 *)ph2c);
+               kfree(ph2c);
                res = _FAIL;
                goto exit;
        }
@@ -1496,7 +1496,7 @@ static int rtw_ap_set_key(
        }
        psetkeyparm = rtw_zmalloc(sizeof(struct setkey_parm));
        if (psetkeyparm == NULL) {
-               kfree((unsigned char *)pcmd);
+               kfree(pcmd);
                res = _FAIL;
                goto exit;
        }

diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c b/drivers/staging/rtl8723bs/core/rtw_cmd.c
index 8d93c2f2689093d8bf68725dbf3aed769ef9aac4..e6fea9665dffc3fad782772f88d3067e238d0d10 100644 (file)
--- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
+++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
@@ -372,13 +372,13 @@ void rtw_free_cmd_obj(struct cmd_obj *pcmd)
        if ((pcmd->cmdcode != _JoinBss_CMD_) &&
            (pcmd->cmdcode != _CreateBss_CMD_)) {
                /* free parmbuf in cmd_obj */
-               kfree((unsigned char *)pcmd->parmbuf);
+               kfree(pcmd->parmbuf);
        }
 
        if (pcmd->rsp != NULL) {
                if (pcmd->rspsz != 0) {
                        /* free rsp in cmd_obj */
-                       kfree((unsigned char *)pcmd->rsp);
+                       kfree(pcmd->rsp);
                }
        }
 

diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c
index 0ac7712223afb60c8f08f115f2fd30167812d97a..1b29a200a4d7e01df6a6837a685429e6f0aff1df 100644 (file)
--- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
+++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
@@ -2207,7 +2207,7 @@ sint rtw_set_auth(struct adapter *adapter, struct security_priv *psecuritypriv)
 
        psetauthparm = rtw_zmalloc(sizeof(struct setauth_parm));
        if (psetauthparm == NULL) {
-               kfree((unsigned char *)pcmd);
+               kfree(pcmd);
                res = _FAIL;
                goto exit;
        }
@@ -2284,7 +2284,7 @@ sint rtw_set_key(struct adapter *adapter, struct security_priv *psecuritypriv, s
        default:
                RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n rtw_set_key:psecuritypriv->dot11PrivacyAlgrthm = %x (must be 1 or 2 or 4 or 5)\n", psecuritypriv->dot11PrivacyAlgrthm));
                res = _FAIL;
-               kfree((unsigned char *)psetkeyparm);
+               kfree(psetkeyparm);
                goto exit;
        }
 
@@ -2292,7 +2292,7 @@ sint rtw_set_key(struct adapter *adapter, struct security_priv *psecuritypriv, s
        if (enqueue) {
                pcmd = rtw_zmalloc(sizeof(struct cmd_obj));
                if (pcmd == NULL) {
-                       kfree((unsigned char *)psetkeyparm);
+                       kfree(psetkeyparm);
                        res = _FAIL;  /* try again */
                        goto exit;
                }
@@ -2308,7 +2308,7 @@ sint rtw_set_key(struct adapter *adapter, struct security_priv *psecuritypriv, s
                res = rtw_enqueue_cmd(pcmdpriv, pcmd);
        } else {
                setkey_hdl(adapter, (u8 *)psetkeyparm);
-               kfree((u8 *) psetkeyparm);
+               kfree(psetkeyparm);
        }
 exit:
        return res;

diff --git a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
index 3933e8637e5770f538598cba009756236693efd5..9590e6f351c1897dd011ec4e44af392ea0cca1ed 100644 (file)
--- a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
+++ b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
@@ -1466,11 +1466,11 @@ int rtw_check_bcn_info(struct adapter *Adapter, u8 *pframe, u32 packet_len)
                }
        }
 
-       kfree((u8 *)bssid);
+       kfree(bssid);
        return _SUCCESS;
 
 _mismatch:
-       kfree((u8 *)bssid);
+       kfree(bssid);
 
        if (pmlmepriv->NumOfBcnInfoChkFail == 0)
                pmlmepriv->timeBcnInfoChkStart = jiffies;

diff --git a/drivers/staging/rtl8723bs/core/rtw_xmit.c b/drivers/staging/rtl8723bs/core/rtw_xmit.c
index 43d663c8486d16d96dd370348e4796decbab4420..f5cb5ae7790b8e00f2f3974ab0bf4e8967eb9635 100644 (file)
--- a/drivers/staging/rtl8723bs/core/rtw_xmit.c
+++ b/drivers/staging/rtl8723bs/core/rtw_xmit.c
@@ -2212,7 +2212,7 @@ void rtw_free_hwxmits(struct adapter *padapter)
 
        hwxmits = pxmitpriv->hwxmits;
        if (hwxmits)
-               kfree((u8 *)hwxmits);
+               kfree(hwxmits);
 }
 
 void rtw_init_hwxmits(struct hw_xmit *phwxmit, sint entry)