Skip to content

Commit 3f320e5

Browse files
tobluxbrauner
authored andcommitted
namespace: Replace simple_strtoul with kstrtoul to parse boot params
Replace simple_strtoul() with the recommended kstrtoul() for parsing the 'mhash_entries=' and 'mphash_entries=' boot parameters. Check the return value of kstrtoul() and reject invalid values. This adds error handling while preserving behavior for existing values, and removes use of the deprecated simple_strtoul() helper. Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev> Link: https://patch.msgid.link/20251214153141.218953-2-thorsten.blum@linux.dev Reviewed-by: Jan Kara <jack@suse.cz> Signed-off-by: Christian Brauner <brauner@kernel.org>
1 parent b29a0a3 commit 3f320e5

1 file changed

Lines changed: 2 additions & 8 deletions

File tree

fs/namespace.c

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -49,20 +49,14 @@ static unsigned int mp_hash_shift __ro_after_init;
4949
static __initdata unsigned long mhash_entries;
5050
static int __init set_mhash_entries(char *str)
5151
{
52-
if (!str)
53-
return 0;
54-
mhash_entries = simple_strtoul(str, &str, 0);
55-
return 1;
52+
return kstrtoul(str, 0, &mhash_entries) == 0;
5653
}
5754
__setup("mhash_entries=", set_mhash_entries);
5855

5956
static __initdata unsigned long mphash_entries;
6057
static int __init set_mphash_entries(char *str)
6158
{
62-
if (!str)
63-
return 0;
64-
mphash_entries = simple_strtoul(str, &str, 0);
65-
return 1;
59+
return kstrtoul(str, 0, &mphash_entries) == 0;
6660
}
6761
__setup("mphash_entries=", set_mphash_entries);
6862

0 commit comments

Comments
 (0)