Re: [PATCH v1 1/1] ACPI: processor: idle: Replace strlcat() with better alternative

[Josh Law]

On 16 March 2026 17:01:58 GMT, Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
>strlcpy() and strlcat() are confusing APIs and the former one already gone
>from the kernel. In preparation to kill strlcat() replace it with the better
>alternative.
>
>Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
>---
> drivers/acpi/processor_idle.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
>diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
>index 45b5d17443cf..498d88f5d5c9 100644
>--- a/drivers/acpi/processor_idle.c
>+++ b/drivers/acpi/processor_idle.c
>@@ -1010,9 +1010,7 @@ static bool combine_lpi_states(struct acpi_lpi_state *local,
> 	result->arch_flags = parent->arch_flags;
> 	result->index = parent->index;
> 
>-	strscpy(result->desc, local->desc, ACPI_CX_DESC_LEN);
>-	strlcat(result->desc, "+", ACPI_CX_DESC_LEN);
>-	strlcat(result->desc, parent->desc, ACPI_CX_DESC_LEN);
>+	snprintf(result->desc, ACPI_CX_DESC_LEN, "%s+%s", local->desc, parent->desc);
> 	return true;
> }
> 


Is this patch in relation to my string.c patch? You would have to do this for about 120~ files.

V/R


Josh Law