summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authortb <>2024-12-04 09:50:52 +0000
committertb <>2024-12-04 09:50:52 +0000
commit54b7e03a99e6dbd79315380653c1bf578c8444b0 (patch)
treef3d93eca2e878b44978a2c46216353762d50271c /src
parentf14751780c164cac61e6f6763b1d8b4e8259080a (diff)
downloadopenbsd-54b7e03a99e6dbd79315380653c1bf578c8444b0.tar.gz
openbsd-54b7e03a99e6dbd79315380653c1bf578c8444b0.tar.bz2
openbsd-54b7e03a99e6dbd79315380653c1bf578c8444b0.zip
Annotate WTLS7 as being wrong
This should really have been using SECP 160R2, not SECP 160R1. Of course this means in particular that nobody ever used this curve, at least not against another implementation than OpenSSL. Quasi-monocultures are poisonous whether the monopolist is benevolent and competent or not.
Diffstat (limited to 'src')
-rw-r--r--src/lib/libcrypto/ec/ec_curve.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/lib/libcrypto/ec/ec_curve.c b/src/lib/libcrypto/ec/ec_curve.c
index fd7831598b..a37f324a24 100644
--- a/src/lib/libcrypto/ec/ec_curve.c
+++ b/src/lib/libcrypto/ec/ec_curve.c
@@ -1,4 +1,4 @@
1/* $OpenBSD: ec_curve.c,v 1.50 2024/11/24 10:12:05 tb Exp $ */ 1/* $OpenBSD: ec_curve.c,v 1.51 2024/12/04 09:50:52 tb Exp $ */
2/* 2/*
3 * Written by Nils Larsch for the OpenSSL project. 3 * Written by Nils Larsch for the OpenSSL project.
4 */ 4 */
@@ -2121,6 +2121,7 @@ static const struct ec_curve {
2121 .cofactor = 1, 2121 .cofactor = 1,
2122 }, 2122 },
2123 { 2123 {
2124 /* XXX - this one's been wrong all along. Should use 160r1. */
2124 .comment = "SECG/WTLS curve over a 160 bit prime field", 2125 .comment = "SECG/WTLS curve over a 160 bit prime field",
2125 .nid = NID_wap_wsg_idm_ecid_wtls7, 2126 .nid = NID_wap_wsg_idm_ecid_wtls7,
2126 .seed_len = sizeof(_EC_SECG_PRIME_160R2.seed), 2127 .seed_len = sizeof(_EC_SECG_PRIME_160R2.seed),