Commit 808bf29b authored by Alexander Sverdlin's avatar Alexander Sverdlin Committed by Linus Torvalds

init: carefully handle loglevel option on kernel cmdline.

When a malformed loglevel value (for example "${abc}") is passed on the
kernel cmdline, the loglevel itself is being set to 0.

That then suppresses all following messages, including all the errors
and crashes caused by other malformed cmdline options.  This could make
debugging process quite tricky.

This patch leaves the previous value of loglevel if the new value is
incorrect and reports an error code in this case.
Signed-off-by: default avatarAlexander Sverdlin <alexander.sverdlin@sysgo.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 32ef4384
...@@ -209,8 +209,19 @@ early_param("quiet", quiet_kernel); ...@@ -209,8 +209,19 @@ early_param("quiet", quiet_kernel);
static int __init loglevel(char *str) static int __init loglevel(char *str)
{ {
get_option(&str, &console_loglevel); int newlevel;
return 0;
/*
* Only update loglevel value when a correct setting was passed,
* to prevent blind crashes (when loglevel being set to 0) that
* are quite hard to debug
*/
if (get_option(&str, &newlevel)) {
console_loglevel = newlevel;
return 0;
}
return -EINVAL;
} }
early_param("loglevel", loglevel); early_param("loglevel", loglevel);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment