From 64762d48ec84d36fc2618920a731368387253efc Mon Sep 17 00:00:00 2001 From: Ovidiu Panait Date: Sun, 7 Jun 2026 00:04:20 +0300 Subject: [PATCH] thermal: sysfs: Replace sscanf() with kstrtoul() Replace sscanf() with kstrtoul() in cur_state_store(), as kstrto is preferred over single-variable sscanf(). Signed-off-by: Ovidiu Panait [ rjw: Changelog edits ] Link: https://patch.msgid.link/20260606210420.2311145-3-ovidiu.panait.oss@gmail.com Signed-off-by: Rafael J. Wysocki --- drivers/thermal/thermal_sysfs.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/thermal/thermal_sysfs.c b/drivers/thermal/thermal_sysfs.c index 9f2f25a6da37..b44abfc997ed 100644 --- a/drivers/thermal/thermal_sysfs.c +++ b/drivers/thermal/thermal_sysfs.c @@ -536,11 +536,9 @@ cur_state_store(struct device *dev, struct device_attribute *attr, unsigned long state; int result; - if (sscanf(buf, "%ld\n", &state) != 1) - return -EINVAL; - - if ((long)state < 0) - return -EINVAL; + result = kstrtoul(buf, 10, &state); + if (result < 0) + return result; /* Requested state should be less than max_state + 1 */ if (state > cdev->max_state)