]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
devtmpfs: Replace simple_strtoul with kstrtoint in mount_param
authorThorsten Blum <thorsten.blum@linux.dev>
Sat, 20 Dec 2025 12:59:31 +0000 (13:59 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 16 Jan 2026 15:44:43 +0000 (16:44 +0100)
Replace simple_strtoul() with the recommended kstrtoint() for parsing
the 'devtmpfs.mount=' boot parameter. Unlike simple_strtoul(), which
returns an unsigned long, kstrtoint() converts the string directly to
int and avoids implicit casting.

Check the return value of kstrtoint() and reject invalid values. This
adds error handling while preserving behavior for existing values, and
removes use of the deprecated simple_strtoul() helper. The current code
silently sets 'mount_dev = 0' if parsing fails, instead of leaving the
default value (IS_ENABLED(CONFIG_DEVTMPFS_MOUNT)) unchanged.

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://patch.msgid.link/20251220125930.76836-2-thorsten.blum@linux.dev
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/base/devtmpfs.c

index 748ad36981070f82786c31bf608515d9d4bfa22c..b1c4ceb65026e689b2dbb43866a4ced0c24dc40f 100644 (file)
@@ -56,8 +56,7 @@ static struct req {
 
 static int __init mount_param(char *str)
 {
-       mount_dev = simple_strtoul(str, NULL, 0);
-       return 1;
+       return kstrtoint(str, 0, &mount_dev) == 0;
 }
 __setup("devtmpfs.mount=", mount_param);