diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2025-12-16 15:52:37 +0100 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2025-12-24 13:49:36 +0100 |
| commit | b29a0a37f46bbfd2a36eff73eb66249d7baaf71a (patch) | |
| tree | 7cb5ce9212a7ae3fca895c8a3f7b44089d5d1e89 | |
| parent | 63ad216fbfe2240da67233e0a0d10af8a12f7bde (diff) | |
dcache: Replace simple_strtoul with kstrtoul in set_dhash_entries
Replace simple_strtoul() with the recommended kstrtoul() for parsing the
'dhash_entries=' boot parameter.
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/20251216145236.44520-2-thorsten.blum@linux.dev
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Christian Brauner <brauner@kernel.org>
| -rw-r--r-- | fs/dcache.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/fs/dcache.c b/fs/dcache.c index dc2fff4811d1..ec275f4fd81c 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@ -3227,10 +3227,7 @@ EXPORT_SYMBOL(d_parent_ino); static __initdata unsigned long dhash_entries; static int __init set_dhash_entries(char *str) { - if (!str) - return 0; - dhash_entries = simple_strtoul(str, &str, 0); - return 1; + return kstrtoul(str, 0, &dhash_entries) == 0; } __setup("dhash_entries=", set_dhash_entries); |