[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aUVI9smf2t7PvDF6@stanley.mountain>
Date: Fri, 19 Dec 2025 15:45:42 +0300
From: Dan Carpenter <dan.carpenter@...aro.org>
To: Daniel Gomez <da.gomez@...nel.org>
Cc: Sami Tolvanen <samitolvanen@...gle.com>,
"Luck, Tony" <tony.luck@...el.com>, Chris Li <sparse@...isli.org>,
Eric Biggers <ebiggers@...nel.org>, Kees Cook <kees@...nel.org>,
Luis Chamberlain <mcgrof@...nel.org>,
Rusty Russell <rusty@...tcorp.com.au>,
Petr Pavlu <petr.pavlu@...e.com>,
"linux-modules@...r.kernel.org" <linux-modules@...r.kernel.org>,
Malcolm Priestley <tvboxspy@...il.com>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Hans Verkuil <hverkuil@...nel.org>,
Uwe Kleine-König <u.kleine-koenig@...gutronix.de>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
"linux-hardening@...r.kernel.org" <linux-hardening@...r.kernel.org>
Subject: Re: [PATCH 3/3] module: Add compile-time check for embedded NUL
characters
On Fri, Dec 12, 2025 at 02:30:48AM +0900, Daniel Gomez wrote:
> Maybe the flag fix just needs to be applied to the evaluation? Other op
> structs do the same. But Dan's patch did not implement evaluate. E.g.:
>
> static struct symbol_op constant_p_op = {
> .evaluate = evaluate_to_int_const_expr,
> .expand = expand_constant_p
> };
>
I was waiting for you to send this as a patch. I can do it if you
need me to.
regards,
dan carpenter
Powered by blists - more mailing lists