[tip: timers/core] clocksource: Remove unnecessary strscpy() size argument

From: tip-bot2 for Thorsten Blum
Date: Thu Mar 13 2025 - 06:59:57 EST


The following commit has been merged into the timers/core branch of tip:

Commit-ID: fc661d0a78673f23a3fd78d0bb20900ee64d1839
Gitweb: https://git.kernel.org/tip/fc661d0a78673f23a3fd78d0bb20900ee64d1839
Author: Thorsten Blum <thorsten.blum@xxxxxxxxx>
AuthorDate: Tue, 11 Mar 2025 12:06:24 +01:00
Committer: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
CommitterDate: Thu, 13 Mar 2025 11:37:44 +01:00

clocksource: Remove unnecessary strscpy() size argument

The size argument of strscpy() is only required when the destination
pointer is not a fixed sized array or when the copy needs to be smaller
than the size of the fixed sized destination array.

For fixed sized destination arrays and full copies, strscpy() automatically
determines the length of the destination buffer if the size argument is
omitted.

This makes the explicit sizeof() unnecessary. Remove it.

[ tglx: Massaged change log ]

Signed-off-by: Thorsten Blum <thorsten.blum@xxxxxxxxx>
Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
Link: https://lore.kernel.org/all/20250311110624.495718-2-thorsten.blum@xxxxxxxxx

---
kernel/time/clocksource.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/time/clocksource.c b/kernel/time/clocksource.c
index 2a7802e..e0eeacb 100644
--- a/kernel/time/clocksource.c
+++ b/kernel/time/clocksource.c
@@ -1510,7 +1510,7 @@ static int __init boot_override_clocksource(char* str)
{
mutex_lock(&clocksource_mutex);
if (str)
- strscpy(override_name, str, sizeof(override_name));
+ strscpy(override_name, str);
mutex_unlock(&clocksource_mutex);
return 1;
}