diff options
author | Krister Johansen <kjlx@templeofstupid.com> | 2023-10-27 14:46:40 -0700 |
---|---|---|
committer | Luis Chamberlain <mcgrof@kernel.org> | 2023-11-01 12:10:02 -0700 |
commit | 8001f49394e353f035306a45bcf504f06fca6355 (patch) | |
tree | 538778c2480a443486588bcc4fd68f33269e9bba /fs/proc/proc_sysctl.c | |
parent | ccee9a2a8c002516d4252952df836abeaddfa39c (diff) | |
download | lwn-8001f49394e353f035306a45bcf504f06fca6355.tar.gz lwn-8001f49394e353f035306a45bcf504f06fca6355.zip |
proc: sysctl: prevent aliased sysctls from getting passed to init
The code that checks for unknown boot options is unaware of the sysctl
alias facility, which maps bootparams to sysctl values. If a user sets
an old value that has a valid alias, a message about an invalid
parameter will be printed during boot, and the parameter will get passed
to init. Fix by checking for the existence of aliased parameters in the
unknown boot parameter code. If an alias exists, don't return an error
or pass the value to init.
Signed-off-by: Krister Johansen <kjlx@templeofstupid.com>
Cc: stable@vger.kernel.org
Fixes: 0a477e1ae21b ("kernel/sysctl: support handling command line aliases")
Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>
Diffstat (limited to 'fs/proc/proc_sysctl.c')
-rw-r--r-- | fs/proc/proc_sysctl.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c index c88854df0b62..1c9635dddb70 100644 --- a/fs/proc/proc_sysctl.c +++ b/fs/proc/proc_sysctl.c @@ -1592,6 +1592,13 @@ static const char *sysctl_find_alias(char *param) return NULL; } +bool sysctl_is_alias(char *param) +{ + const char *alias = sysctl_find_alias(param); + + return alias != NULL; +} + /* Set sysctl value passed on kernel command line. */ static int process_sysctl_arg(char *param, char *val, const char *unused, void *arg) |