diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2025-09-18 18:24:47 +0200 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2025-10-31 10:12:32 +0100 |
| commit | b2c43efc3c8d9cda34eb8421249d15e6fed4d650 (patch) | |
| tree | 79af29e13ae947fd80d2832a43fe0020f26d3b8f | |
| parent | 3a8660878839faadb4f1a6dd72c3179c1df56787 (diff) | |
initrd: Replace simple_strtol with kstrtoint to improve ramdisk_start_setup
Replace simple_strtol() with the recommended kstrtoint() for parsing the
'ramdisk_start=' boot parameter. Unlike simple_strtol(), which returns a
a long, kstrtoint() converts the string directly to an integer and
avoids implicit casting.
Check the return value of kstrtoint() and reject invalid values. This
adds error handling while preserving existing behavior for valid values,
and removes use of the deprecated simple_strtol() helper.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Christian Brauner <brauner@kernel.org>
| -rw-r--r-- | init/do_mounts_rd.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/init/do_mounts_rd.c b/init/do_mounts_rd.c index 19d9f33dcacf..eddbe5cb0413 100644 --- a/init/do_mounts_rd.c +++ b/init/do_mounts_rd.c @@ -29,8 +29,7 @@ int __initdata rd_image_start; /* starting block # of image */ static int __init ramdisk_start_setup(char *str) { - rd_image_start = simple_strtol(str,NULL,0); - return 1; + return kstrtoint(str, 0, &rd_image_start) == 0; } __setup("ramdisk_start=", ramdisk_start_setup); |