linux-kernel - Re: [PATCH] KEYS: include header for EINVAL definition

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]

Message-Id: <CYDVFYGDLJFC.1IAQAPLY6VRWX@kernel.org>
Date: Sat, 13 Jan 2024 22:46:57 +0200
From: "Jarkko Sakkinen" <jarkko@...nel.org>
To: "Clay Chang" <clayc@....com>, <keyrings@...r.kernel.org>,
 <linux-crypto@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Cc: <dhowells@...hat.com>, <herbert@...dor.apana.org.au>,
 <davem@...emloft.net>
Subject: Re: [PATCH] KEYS: include header for EINVAL definition

On Fri Jan 12, 2024 at 7:59 AM EET, Clay Chang wrote:
> Hi Jarkko,
>
> Yes, I have. I am working on an OpenBMC project and the kernel .config was generated by merging several config fragments. In one experiment where the CONFIG_ASYMMETRIC_PUBLIC_KEY_SUBTYPE was not presented, I got a EINVAL undeclared compilation error.
>
> Do you want me to show the full .config here?

Well the problem with the patch is that it does not *prove*
that there is issue.

So you evidence in the commit message that there is bug in
the first place and which config options cause it. And it
it must be testable with latest mainline (not by downstream
project).

BR, Jarkko