Re: [PATCH V4] staging: r8188eu: Remove broken rtw_p2p_get function.

From: Dan Carpenter
Date: Tue May 03 2022 - 07:37:33 EST


On Sat, Apr 30, 2022 at 12:28:59AM -0400, Charlie Sands wrote:
> Removes broken and unsafe rtw_p2p_get function and all of the code
> associated with it from the r8188eu driver.
>
> Signed-off-by: Charlie Sands <charlies256@xxxxxxxxxxxxxx>
> ---
>
> Follow up to the patch called "staging: r8188eu: Fix unsafe memory access
> by memcmp."
>
> V2: Fixed checkpatch.pl warning and changed variable name as suggested
> by Greg K. H. and improved error checking on the "copy_from_user" function as
> suggested by Pavel Skripkin.
>
> V3: Return -EFAULT from the function when there is a memory error while
> moving it from user space to kernel space and fix incorrect subject from V2
>
> V4: Updated commit message and changes to reflect those sugested by Dan
> Carpenter.
>

Thanks!

Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

regards,
dan carpenter