[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <5cb4ed29-0039-432a-be2f-629aa7e1278c@app.fastmail.com>
Date: Thu, 10 Oct 2024 12:10:48 +1300
From: "Luke Jones" <luke@...nes.dev>
To: "Abdul Rahim" <abdul.rahim@...ahoo.com>, corentin.chary@...il.com,
"Hans de Goede" <hdegoede@...hat.com>,
Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
Cc: platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] asus-laptop: prefer strscpy() over strcpy()
On Thu, 10 Oct 2024, at 12:05 PM, Abdul Rahim wrote:
> The function strcpy() is depreciated and potentially unsafe. It performs
> no bounds checking on the destination buffer. This could result in
> linear overflows beyond the end of the buffer, leading to all kinds of
> misbehaviors. The safe replacement is strscpy() [1].
>
> this fixes checkpatch warning:
> WARNING: Prefer strscpy over strcpy
>
> Link: https://www.kernel.org/doc/html/latest/process/deprecated.html#strcpy [1]
> Signed-off-by: Abdul Rahim <abdul.rahim@...ahoo.com>
> ---
> drivers/platform/x86/asus-laptop.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/platform/x86/asus-laptop.c
> b/drivers/platform/x86/asus-laptop.c
> index 9d7e6b712abf..d460dd194f19 100644
> --- a/drivers/platform/x86/asus-laptop.c
> +++ b/drivers/platform/x86/asus-laptop.c
> @@ -1832,8 +1832,8 @@ static int asus_acpi_add(struct acpi_device
> *device)
> if (!asus)
> return -ENOMEM;
> asus->handle = device->handle;
> - strcpy(acpi_device_name(device), ASUS_LAPTOP_DEVICE_NAME);
> - strcpy(acpi_device_class(device), ASUS_LAPTOP_CLASS);
> + strscpy(acpi_device_name(device), ASUS_LAPTOP_DEVICE_NAME);
> + strscpy(acpi_device_class(device), ASUS_LAPTOP_CLASS);
> device->driver_data = asus;
> asus->device = device;
>
> --
> 2.43.0
Looks good, thank you for doing this.
Reviewed-by: Luke D. Jones <luke@...nes.dev>
Powered by blists - more mailing lists