lkml.org 
[lkml]   [2022]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] drivers: usbtouchscreen: use kzalloc
On Sat, Mar 12, 2022 at 06:22:30PM +0100, Michael Estner wrote:
> Use kzalloc instead of kmalloc + memset.
>
> The semantic patch that makes this change is:
> (https://coccinelle.gitlabpages.inria.fr/website/)
>
> Signed-off-by: Michael Estner <michaelestner@web.de>
> ---
> drivers/input/touchscreen/usbtouchscreen.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/input/touchscreen/usbtouchscreen.c b/drivers/input/touchscreen/usbtouchscreen.c
> index 43c521f50c85..159e1ac6198a 100644
> --- a/drivers/input/touchscreen/usbtouchscreen.c
> +++ b/drivers/input/touchscreen/usbtouchscreen.c
> @@ -975,7 +975,7 @@ static int nexio_init(struct usbtouch_usb *usbtouch)
> if (!input_ep || !output_ep)
> return -ENXIO;
>
> - buf = kmalloc(NEXIO_BUFSIZE, GFP_NOIO);
> + buf = kzalloc(NEXIO_BUFSIZE, GFP_NOIO);
> if (!buf)
> goto out_buf;
>
> @@ -998,7 +998,6 @@ static int nexio_init(struct usbtouch_usb *usbtouch)
>
> /* read replies */
> for (i = 0; i < 3; i++) {
> - memset(buf, 0, NEXIO_BUFSIZE);
> ret = usb_bulk_msg(dev, usb_rcvbulkpipe(dev, input_ep),
> buf, NEXIO_BUFSIZE, &actual_len,
> NEXIO_TIMEOUT);

NAK

The new code is not equivalent to the old so that semantic patch must be
broken.

The current code is clearing the buffer before each read, after this
patch it would only be cleared once.

Johan

\
 
 \ /
  Last update: 2022-03-12 19:15    [W:0.030 / U:2.472 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site