From: Gustavo A. R. Silva Date: Mon, 22 Apr 2019 16:39:34 +0000 (-0500) Subject: ACPI: event: replace strcpy() by strscpy() X-Git-Tag: v5.2-rc1~183^2~2^4~1 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c7d5f21e8d184f255785e02760139037a97fd796;p=thirdparty%2Flinux.git ACPI: event: replace strcpy() by strscpy() The strcpy() function is being deprecated. Replace it by the safer strscpy() and fix the following Coverity warnings: "You might overrun the 15-character fixed-size string event->bus_id by copying bus_id without checking the length." "You might overrun the 20-character fixed-size string event->device_class by copying device_class without checking the length." Addresses-Coverity-ID: 139001 ("Copy into fixed size buffer") Signed-off-by: Gustavo A. R. Silva Signed-off-by: Rafael J. Wysocki --- diff --git a/drivers/acpi/event.c b/drivers/acpi/event.c index 5a127f3f2d5c3..47f21599f2ab5 100644 --- a/drivers/acpi/event.c +++ b/drivers/acpi/event.c @@ -131,8 +131,8 @@ int acpi_bus_generate_netlink_event(const char *device_class, event = nla_data(attr); memset(event, 0, sizeof(struct acpi_genl_event)); - strcpy(event->device_class, device_class); - strcpy(event->bus_id, bus_id); + strscpy(event->device_class, device_class, sizeof(event->device_class)); + strscpy(event->bus_id, bus_id, sizeof(event->bus_id)); event->type = type; event->data = data;