Re: [PATCH] params: Fix an overflow in param_attr_show
From: Jean Delvare
Date: Thu Sep 28 2017 - 12:01:33 EST
On Thu, 28 Sep 2017 10:38:27 +0200, Ingo Molnar wrote:
> * Jean Delvare <jdelvare@xxxxxxx> wrote:
> > Or... I could append the \n inside the STANDARD_PARAM_DEF macro, so the
> > calls are unchanged. Makes my patch smaller, and addresses your concern
> > just as well, I suppose.
>
> Yeah, that would be even better:
>
> Acked-by: Ingo Molnar <mingo@xxxxxxxxxx>
>
> Note that the vertical alignment makes things easier to read regardless of the \n.
Agreed, but now I'll make it a separate patch.
--
Jean Delvare
SUSE L3 Support