The way the comments in the *switch* statement in module_sig_check() are indented, it may seem they refer to the statements above them, not below. Align the comments with the *case* and *default* labels below them, fixing the comment style and adding article/dash, while at it...
Signed-off-by: Sergey Shtylyov <s.shtyl...@omprussia.ru> --- kernel/module.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) Index: linux/kernel/module.c =================================================================== --- linux.orig/kernel/module.c +++ linux/kernel/module.c @@ -2901,10 +2901,11 @@ static int module_sig_check(struct load_ info->sig_ok = true; return 0; - /* We don't permit modules to be loaded into trusted kernels - * without a valid signature on them, but if we're not - * enforcing, certain errors are non-fatal. - */ + /* + * We don't permit modules to be loaded into the trusted kernels + * without a valid signature on them, but if we're not enforcing, + * certain errors are non-fatal. + */ case -ENODATA: reason = "no signature"; goto decide; @@ -2922,10 +2923,10 @@ static int module_sig_check(struct load_ return security_locked_down(LOCKDOWN_MODULE_SIGNATURE); - /* All other errors are fatal, including nomem, unparseable - * signatures and signature check failures - even if signatures - * aren't required. - */ + /* + * All other errors are fatal, including nomem, unparseable signatures + * and signature check failures -- even if signatures aren't required. + */ default: return err; }