diff options
| author | Kari Argillander <kari.argillander@gmail.com> | 2025-12-19 18:48:14 +0300 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2026-01-31 18:38:31 +0300 |
| commit | aa8126b914a767ccacaad53596193d4b97abcd58 (patch) | |
| tree | 44da3106c1b3afe671ded196f32af470734d0f2b | |
| parent | 9ef5c1065cd12e2288c2185f88da5cffce0a4623 (diff) | |
| download | linux-aa8126b914a767ccacaad53596193d4b97abcd58.tar.xz | |
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 <kari.argillander@gmail.com>
Link: https://lore.kernel.org/r/20251219-hwmon-fixes-v1-1-21b29097ea3b@gmail.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
| -rw-r--r-- | drivers/hwmon/hwmon.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/hwmon/hwmon.c b/drivers/hwmon/hwmon.c index 0b4bdcd33c7b..7a784d685d77 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); |