init: carefully handle loglevel option on kernel cmdline.
authorAlexander Sverdlin <alexander.sverdlin@sysgo.com>
Wed, 21 Sep 2011 07:51:40 +0000 (09:51 +0200)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 21 Sep 2011 20:18:52 +0000 (13:18 -0700)
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: Alexander Sverdlin <alexander.sverdlin@sysgo.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
init/main.c

index 9c51ee7adf3d7edb01bfc501488389d4ca1c45fc..2a9b88aa5e7601f83031ed5623dbbb25742dad30 100644 (file)
@@ -209,8 +209,19 @@ early_param("quiet", quiet_kernel);
 
 static int __init loglevel(char *str)
 {
-       get_option(&str, &console_loglevel);
-       return 0;
+       int newlevel;
+
+       /*
+        * 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);