]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
crypto: mscode_parser - remove sha224 authenticode support
authorDimitri John Ledkov <dimitri.ledkov@canonical.com>
Tue, 10 Oct 2023 21:25:29 +0000 (22:25 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 20 Oct 2023 05:39:26 +0000 (13:39 +0800)
It is possible to stand up own certificates and sign PE-COFF binaries
using SHA-224. However it never became popular or needed since it has
similar costs as SHA-256. Windows Authenticode infrastructure never
had support for SHA-224, and all secureboot keys used fro linux
vmlinuz have always been using at least SHA-256.

Given the point of mscode_parser is to support interoperatiblity with
typical de-facto hashes, remove support for SHA-224 to avoid
posibility of creating interoperatibility issues with rhboot/shim,
grub, and non-linux systems trying to sign or verify vmlinux.

SHA-224 itself is not removed from the kernel, as it is truncated
SHA-256. If requested I can write patches to remove SHA-224 support
across all of the drivers.

Signed-off-by: Dimitri John Ledkov <dimitri.ledkov@canonical.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
crypto/asymmetric_keys/mscode_parser.c

index 6416bded0e073f2858ffea1346ddf729aabbb022..855cbc46a9c3689866846ba16f13b57bc16ad4fc 100644 (file)
@@ -84,9 +84,6 @@ int mscode_note_digest_algo(void *context, size_t hdrlen,
        case OID_sha512:
                ctx->digest_algo = "sha512";
                break;
-       case OID_sha224:
-               ctx->digest_algo = "sha224";
-               break;
 
        case OID__NR:
                sprint_oid(value, vlen, buffer, sizeof(buffer));