OSDN Git Service

thermal/drivers/armada: Use strscpy() to instead of strncpy()
authorXu Panda <xu.panda@zte.com.cn>
Wed, 28 Dec 2022 01:45:49 +0000 (09:45 +0800)
committerDaniel Lezcano <daniel.lezcano@kernel.org>
Fri, 6 Jan 2023 13:14:48 +0000 (14:14 +0100)
The implementation of strscpy() is more robust and safer.
That's now the recommended way to copy NUL-terminated strings.

Signed-off-by: Xu Panda <xu.panda@zte.com.cn>
Signed-off-by: Yang Yang <yang.yang29@zte.com>
Link: https://lore.kernel.org/r/202212280945491860150@zte.com.cn
Signed-off-by: Daniel Lezcano <daniel.lezcano@kernel.org>
drivers/thermal/armada_thermal.c

index b28695a..db040db 100644 (file)
@@ -761,8 +761,7 @@ static void armada_set_sane_name(struct platform_device *pdev,
        }
 
        /* Save the name locally */
-       strncpy(priv->zone_name, name, THERMAL_NAME_LENGTH - 1);
-       priv->zone_name[THERMAL_NAME_LENGTH - 1] = '\0';
+       strscpy(priv->zone_name, name, THERMAL_NAME_LENGTH);
 
        /* Then check there are no '-' or hwmon core will complain */
        do {