Skip to content

Commit 8bbdeb7

Browse files
committed
pkcs7, x509: Add ML-DSA support
Add support for ML-DSA keys and signatures to the CMS/PKCS#7 and X.509 implementations. ML-DSA-44, -65 and -87 are all supported. For X.509 certificates, the TBSCertificate is required to be signed directly; for CMS, direct signing of the data is preferred, though use of SHA512 (and only that) as an intermediate hash of the content is permitted with signedAttrs. Signed-off-by: David Howells <dhowells@redhat.com> cc: Lukas Wunner <lukas@wunner.de> cc: Ignat Korchagin <ignat@cloudflare.com> cc: Stephan Mueller <smueller@chronox.de> cc: Eric Biggers <ebiggers@kernel.org> cc: Herbert Xu <herbert@gondor.apana.org.au> cc: keyrings@vger.kernel.org cc: linux-crypto@vger.kernel.org
1 parent f3eccec commit 8bbdeb7

4 files changed

Lines changed: 64 additions & 2 deletions

File tree

crypto/asymmetric_keys/pkcs7_parser.c

Lines changed: 23 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,11 +95,18 @@ static int pkcs7_check_authattrs(struct pkcs7_message *msg)
9595
if (sinfo->authattrs) {
9696
want = true;
9797
msg->have_authattrs = true;
98+
} else if (sinfo->sig->algo_takes_data) {
99+
sinfo->sig->hash_algo = "none";
98100
}
99101

100-
for (sinfo = sinfo->next; sinfo; sinfo = sinfo->next)
102+
for (sinfo = sinfo->next; sinfo; sinfo = sinfo->next) {
101103
if (!!sinfo->authattrs != want)
102104
goto inconsistent;
105+
106+
if (!sinfo->authattrs &&
107+
sinfo->sig->algo_takes_data)
108+
sinfo->sig->hash_algo = "none";
109+
}
103110
return 0;
104111

105112
inconsistent:
@@ -297,6 +304,21 @@ int pkcs7_sig_note_pkey_algo(void *context, size_t hdrlen,
297304
ctx->sinfo->sig->pkey_algo = "ecrdsa";
298305
ctx->sinfo->sig->encoding = "raw";
299306
break;
307+
case OID_id_ml_dsa_44:
308+
ctx->sinfo->sig->pkey_algo = "mldsa44";
309+
ctx->sinfo->sig->encoding = "raw";
310+
ctx->sinfo->sig->algo_takes_data = true;
311+
break;
312+
case OID_id_ml_dsa_65:
313+
ctx->sinfo->sig->pkey_algo = "mldsa65";
314+
ctx->sinfo->sig->encoding = "raw";
315+
ctx->sinfo->sig->algo_takes_data = true;
316+
break;
317+
case OID_id_ml_dsa_87:
318+
ctx->sinfo->sig->pkey_algo = "mldsa87";
319+
ctx->sinfo->sig->encoding = "raw";
320+
ctx->sinfo->sig->algo_takes_data = true;
321+
break;
300322
default:
301323
printk("Unsupported pkey algo: %u\n", ctx->last_oid);
302324
return -ENOPKG;

crypto/asymmetric_keys/public_key.c

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,16 @@ software_key_determine_akcipher(const struct public_key *pkey,
142142
if (strcmp(hash_algo, "streebog256") != 0 &&
143143
strcmp(hash_algo, "streebog512") != 0)
144144
return -EINVAL;
145+
} else if (strcmp(pkey->pkey_algo, "mldsa44") == 0 ||
146+
strcmp(pkey->pkey_algo, "mldsa65") == 0 ||
147+
strcmp(pkey->pkey_algo, "mldsa87") == 0) {
148+
if (strcmp(encoding, "raw") != 0)
149+
return -EINVAL;
150+
if (!hash_algo)
151+
return -EINVAL;
152+
if (strcmp(hash_algo, "none") != 0 &&
153+
strcmp(hash_algo, "sha512") != 0)
154+
return -EINVAL;
145155
} else {
146156
/* Unknown public key algorithm */
147157
return -ENOPKG;

crypto/asymmetric_keys/x509_cert_parser.c

Lines changed: 26 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -257,6 +257,15 @@ int x509_note_sig_algo(void *context, size_t hdrlen, unsigned char tag,
257257
case OID_gost2012Signature512:
258258
ctx->cert->sig->hash_algo = "streebog512";
259259
goto ecrdsa;
260+
case OID_id_ml_dsa_44:
261+
ctx->cert->sig->pkey_algo = "mldsa44";
262+
goto ml_dsa;
263+
case OID_id_ml_dsa_65:
264+
ctx->cert->sig->pkey_algo = "mldsa65";
265+
goto ml_dsa;
266+
case OID_id_ml_dsa_87:
267+
ctx->cert->sig->pkey_algo = "mldsa87";
268+
goto ml_dsa;
260269
}
261270

262271
rsa_pkcs1:
@@ -274,6 +283,12 @@ int x509_note_sig_algo(void *context, size_t hdrlen, unsigned char tag,
274283
ctx->cert->sig->encoding = "x962";
275284
ctx->sig_algo = ctx->last_oid;
276285
return 0;
286+
ml_dsa:
287+
ctx->cert->sig->algo_takes_data = true;
288+
ctx->cert->sig->hash_algo = "none";
289+
ctx->cert->sig->encoding = "raw";
290+
ctx->sig_algo = ctx->last_oid;
291+
return 0;
277292
}
278293

279294
/*
@@ -300,7 +315,8 @@ int x509_note_signature(void *context, size_t hdrlen,
300315

301316
if (strcmp(ctx->cert->sig->pkey_algo, "rsa") == 0 ||
302317
strcmp(ctx->cert->sig->pkey_algo, "ecrdsa") == 0 ||
303-
strcmp(ctx->cert->sig->pkey_algo, "ecdsa") == 0) {
318+
strcmp(ctx->cert->sig->pkey_algo, "ecdsa") == 0 ||
319+
strncmp(ctx->cert->sig->pkey_algo, "mldsa", 5) == 0) {
304320
/* Discard the BIT STRING metadata */
305321
if (vlen < 1 || *(const u8 *)value != 0)
306322
return -EBADMSG;
@@ -524,6 +540,15 @@ int x509_extract_key_data(void *context, size_t hdrlen,
524540
return -ENOPKG;
525541
}
526542
break;
543+
case OID_id_ml_dsa_44:
544+
ctx->cert->pub->pkey_algo = "mldsa44";
545+
break;
546+
case OID_id_ml_dsa_65:
547+
ctx->cert->pub->pkey_algo = "mldsa65";
548+
break;
549+
case OID_id_ml_dsa_87:
550+
ctx->cert->pub->pkey_algo = "mldsa87";
551+
break;
527552
default:
528553
return -ENOPKG;
529554
}

include/linux/oid_registry.h

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,11 @@ enum OID {
145145
OID_id_rsassa_pkcs1_v1_5_with_sha3_384, /* 2.16.840.1.101.3.4.3.15 */
146146
OID_id_rsassa_pkcs1_v1_5_with_sha3_512, /* 2.16.840.1.101.3.4.3.16 */
147147

148+
/* NIST FIPS-204 ML-DSA */
149+
OID_id_ml_dsa_44, /* 2.16.840.1.101.3.4.3.17 */
150+
OID_id_ml_dsa_65, /* 2.16.840.1.101.3.4.3.18 */
151+
OID_id_ml_dsa_87, /* 2.16.840.1.101.3.4.3.19 */
152+
148153
OID__NR
149154
};
150155

0 commit comments

Comments
 (0)