linux-kernel - Re: [PATCH 1/1] params: don't ignore the rest of cmdline if parse

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 26 Aug 2015 09:43:01 +0930
From:	Rusty Russell <rusty@...tcorp.com.au>
To:	Oleg Nesterov <oleg@...hat.com>
Cc:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>,
	Ingo Molnar <mingo@...hat.com>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Peter Zijlstra <peterz@...radead.org>,
	Tejun Heo <tj@...nel.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] params: don't ignore the rest of cmdline if parse_one() fails

Oleg Nesterov <oleg@...hat.com> writes:
> parse_args() just aborts after it hits an error, so other args
> at the same initcall level are simply ignored. This can lead to
> other hard-to-understand problems, for example my testing machine
> panics during the boot if I pass "locktorture.verbose=true".
>
> Change parse_args() to save the err code for return and continue.
>
> Signed-off-by: Oleg Nesterov <oleg@...hat.com>

Thanks!

Applied,
Rusty.

> ---
>  kernel/params.c |   17 +++++++++--------
>  1 files changed, 9 insertions(+), 8 deletions(-)
>
> diff --git a/kernel/params.c b/kernel/params.c
> index a22d6a7..b21139f 100644
> --- a/kernel/params.c
> +++ b/kernel/params.c
> @@ -223,7 +223,7 @@ char *parse_args(const char *doing,
>  		 int (*unknown)(char *param, char *val,
>  				const char *doing, void *arg))
>  {
> -	char *param, *val;
> +	char *param, *val, *err = NULL;
>  
>  	/* Chew leading spaces */
>  	args = skip_spaces(args);
> @@ -238,7 +238,7 @@ char *parse_args(const char *doing,
>  		args = next_arg(args, &param, &val);
>  		/* Stop at -- */
>  		if (!val && strcmp(param, "--") == 0)
> -			return args;
> +			return err ?: args;
>  		irq_was_disabled = irqs_disabled();
>  		ret = parse_one(param, val, doing, params, num,
>  				min_level, max_level, arg, unknown);
> @@ -247,24 +247,25 @@ char *parse_args(const char *doing,
>  				doing, param);
>  
>  		switch (ret) {
> +		case 0:
> +			continue;
>  		case -ENOENT:
>  			pr_err("%s: Unknown parameter `%s'\n", doing, param);
> -			return ERR_PTR(ret);
> +			break;
>  		case -ENOSPC:
>  			pr_err("%s: `%s' too large for parameter `%s'\n",
>  			       doing, val ?: "", param);
> -			return ERR_PTR(ret);
> -		case 0:
>  			break;
>  		default:
>  			pr_err("%s: `%s' invalid for parameter `%s'\n",
>  			       doing, val ?: "", param);
> -			return ERR_PTR(ret);
> +			break;
>  		}
> +
> +		err = ERR_PTR(ret);
>  	}
>  
> -	/* All parsed OK. */
> -	return NULL;
> +	return err;
>  }
>  
>  /* Lazy bastard, eh? */
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/