Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 428715ba authored by Amitoj Kaur Chawla's avatar Amitoj Kaur Chawla Committed by Greg Kroah-Hartman
Browse files

staging: rtl8723bs: Remove unnecessary cast in kfree 



Remove unnecassary casts in the argument to kfree.

Found using Coccinelle. The semantic patch used to find this is as
follows:

//<smpl>
@@
type T;
expression *f;
@@

- kfree((T *)(f));
+ kfree(f);
//</smpl>

Signed-off-by: default avatarAmitoj Kaur Chawla <amitoj1606@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 7bb0dffd

drivers/staging/rtl8723bs/os_dep/ioctl_linux.c

+7 −7
Original line number Original line Diff line number Diff line
@@ -766,7 +766,7 @@ static int wpa_set_encryption(struct net_device *dev, struct ieee_param *param, 




exit:

exit:





	kfree((u8 *)pwep);

	kfree(pwep);

	return ret;

	return ret;

}

}
@@ -2500,7 +2500,7 @@ static int rtw_wx_set_enc_ext(struct net_device *dev, 
	ret =  wpa_set_encryption(dev, param, param_len);

	ret =  wpa_set_encryption(dev, param, param_len);





exit:

exit:

	kfree((u8 *)param);

	kfree(param);





	return ret;

	return ret;

}

}
@@ -3767,7 +3767,7 @@ static int wpa_supplicant_ioctl(struct net_device *dev, struct iw_point *p) 




	if (copy_from_user(param, p->pointer, p->length))

	if (copy_from_user(param, p->pointer, p->length))

	{

	{

		kfree((u8 *)param);

		kfree(param);

		ret = -EFAULT;

		ret = -EFAULT;

		goto out;

		goto out;

	}

	}
@@ -3801,7 +3801,7 @@ static int wpa_supplicant_ioctl(struct net_device *dev, struct iw_point *p) 
	if (ret == 0 && copy_to_user(p->pointer, param, p->length))

	if (ret == 0 && copy_to_user(p->pointer, param, p->length))

		ret = -EFAULT;

		ret = -EFAULT;





	kfree((u8 *)param);

	kfree(param);





out:

out:
@@ -4130,7 +4130,7 @@ static int rtw_set_encryption(struct net_device *dev, struct ieee_param *param, 
	}

	}





exit:

exit:

	kfree((u8 *)pwep);

	kfree(pwep);





	return ret;

	return ret;
@@ -4713,7 +4713,7 @@ static int rtw_hostapd_ioctl(struct net_device *dev, struct iw_point *p) 




	if (copy_from_user(param, p->pointer, p->length))

	if (copy_from_user(param, p->pointer, p->length))

	{

	{

		kfree((u8 *)param);

		kfree(param);

		ret = -EFAULT;

		ret = -EFAULT;

		goto out;

		goto out;

	}

	}
@@ -4817,7 +4817,7 @@ static int rtw_hostapd_ioctl(struct net_device *dev, struct iw_point *p) 
		ret = -EFAULT;

		ret = -EFAULT;









	kfree((u8 *)param);

	kfree(param);





out:

out: