diff options
| author | Abdul Rahim <abdul.rahim@myyahoo.com> | 2024-09-14 02:41:35 +0530 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2024-11-05 21:21:28 +0100 |
| commit | 95504d54a2751ad3e995c7bbafc2116affb28ab9 (patch) | |
| tree | 7d2e27dfae4ade64e758f2701b43d732ab05f4fa | |
| parent | 59b723cd2adbac2a34fc8e12c74ae26ae45bf230 (diff) | |
| download | lwn-95504d54a2751ad3e995c7bbafc2116affb28ab9.tar.gz lwn-95504d54a2751ad3e995c7bbafc2116affb28ab9.zip | |
ACPI: thermal: Use strscpy() instead of strcpy()
strcpy() is generally considered unsafe and use of strscpy() is
recommended [1].
Also using strscpy() instead of strcpy() makes the following checkpatch
warning go away:
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@myyahoo.com>
Link: https://patch.msgid.link/20240913211156.103864-1-abdul.rahim@myyahoo.com
[ rjw: Changelog edits ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
| -rw-r--r-- | drivers/acpi/thermal.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/acpi/thermal.c b/drivers/acpi/thermal.c index 78db38c7076e..6671537cb4b7 100644 --- a/drivers/acpi/thermal.c +++ b/drivers/acpi/thermal.c @@ -796,9 +796,9 @@ static int acpi_thermal_add(struct acpi_device *device) return -ENOMEM; tz->device = device; - strcpy(tz->name, device->pnp.bus_id); - strcpy(acpi_device_name(device), ACPI_THERMAL_DEVICE_NAME); - strcpy(acpi_device_class(device), ACPI_THERMAL_CLASS); + strscpy(tz->name, device->pnp.bus_id); + strscpy(acpi_device_name(device), ACPI_THERMAL_DEVICE_NAME); + strscpy(acpi_device_class(device), ACPI_THERMAL_CLASS); device->driver_data = tz; acpi_thermal_aml_dependency_fix(tz); |