linux-kernel - RE: Accidental config renaming in commit 3f342a23257d ("crypto: Kconfig

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 for Android: free password hash cracker in your pocket

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

Date:   Tue, 13 Sep 2022 08:30:45 +0000
From:   "Elliott, Robert (Servers)" <elliott@....com>
To:     Lukas Bulwahn <lukas.bulwahn@...il.com>,
        Herbert Xu <herbert@...dor.apana.org.au>,
        "linux-crypto@...r.kernel.org" <linux-crypto@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: RE: Accidental config renaming in commit 3f342a23257d ("crypto:
 Kconfig - simplify hash entries")

Sorry for the delay - I'm out of office this month.

1. I agree this is wrong:

-config CRYPTO_SHA1_ARM64_CE
-	tristate "SHA-1 digest algorithm (ARMv8 Crypto Extensions)"
+config CRYPTO_SHA1_ARM64
+	tristate "Hash functions: SHA-1 (ARMv8 Crypto Extensions)"


I think my grep comparisons to make sure the config lines didn't
change were foiled by - being a special character.

Herbert, can you fix that in a way that won't
cause bisection anomalies?

2. There's one more problem in that commit - the
CRYPTO_BLAKE2S symbol was dropped by 6.0 and should no
longer exist. It's a remnant from the original series
being based on 5.19.

The extra space before "config" led me to overlook it.
(I'll investigate if checkpatch.pl can check for leading
spaces before keywords like this next month)

+ config CRYPTO_BLAKE2S
+	tristate "BLAKE2s"
+        select CRYPTO_LIB_BLAKE2S_GENERIC
+        select CRYPTO_HASH
+        help
+	  BLAKE2s cryptographic hash function (RFC 7693)
+
+	  BLAKE2s is optimized for 8 to 32-bit platforms and can produce
+	  digests of any size between 1 and 32 bytes. The keyed hash is
+	  also implemented.
+
+	  This module provides the following algorithms:
+          - blake2s-128
+          - blake2s-160
+          - blake2s-224
+          - blake2s-256
+
+	  Used by Wireguard.
+
+          See https://blake2.net for further information.



> -----Original Message-----
> From: Lukas Bulwahn <lukas.bulwahn@...il.com>
> Sent: Tuesday, September 6, 2022 9:28 PM
> To: Elliott, Robert (Servers) <elliott@....com>; Herbert Xu
> <herbert@...dor.apana.org.au>; linux-crypto@...r.kernel.org; Linux Kernel
> Mailing List <linux-kernel@...r.kernel.org>
> Subject: Accidental config renaming in commit 3f342a23257d ("crypto:
> Kconfig - simplify hash entries")
> 
> Dear Robert,
> 
> I noticed that:
> 
> Commit 3f342a23257d ("crypto: Kconfig - simplify hash entries") makes
> a lot of changes to the config descriptions, but among all those
> changes, it also renames CRYPTO_SHA1_ARM64_CE to CRYPTO_SHA1_ARM64.
> 
> Given that you did not touch the corresponding Makefile, it seems that
> this config renaming was unintended and accidentally.
> 
> Could you please confirm that?
> 
> Best regards,
> 
> Lukas