From aa8126b914a767ccacaad53596193d4b97abcd58 Mon Sep 17 00:00:00 2001 From: Kari Argillander Date: Fri, 19 Dec 2025 17:48:14 +0200 Subject: [PATCH] hwmon: Fix wrong return errno in *sanitize_name() Currently if user of *sanitize_name() function gives nullptr for name they get's ENOMEM. Logically it should be EINVAL. Signed-off-by: Kari Argillander Link: https://lore.kernel.org/r/20251219-hwmon-fixes-v1-1-21b29097ea3b@gmail.com Signed-off-by: Guenter Roeck --- drivers/hwmon/hwmon.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/hwmon/hwmon.c b/drivers/hwmon/hwmon.c index 0b4bdcd33c7b8..7a784d685d77f 100644 --- a/drivers/hwmon/hwmon.c +++ b/drivers/hwmon/hwmon.c @@ -1260,6 +1260,9 @@ static char *__hwmon_sanitize_name(struct device *dev, const char *old_name) */ char *hwmon_sanitize_name(const char *name) { + if (!name) + return ERR_PTR(-EINVAL); + return __hwmon_sanitize_name(NULL, name); } EXPORT_SYMBOL_GPL(hwmon_sanitize_name); @@ -1276,7 +1279,7 @@ EXPORT_SYMBOL_GPL(hwmon_sanitize_name); */ char *devm_hwmon_sanitize_name(struct device *dev, const char *name) { - if (!dev) + if (!dev || !name) return ERR_PTR(-EINVAL); return __hwmon_sanitize_name(dev, name); -- 2.47.3