diff options
| author | jsing <> | 2018-06-13 15:13:30 +0000 |
|---|---|---|
| committer | jsing <> | 2018-06-13 15:13:30 +0000 |
| commit | d579103b8da90de816647e4df81e92ed59fcff8f (patch) | |
| tree | 443a17c6ed4157f4256222f1c1ddae11c9d024bf /src/lib | |
| parent | 752cb6f550d07071e7ee95a9266cb5f555f57ee6 (diff) | |
| download | openbsd-libressl-v2.7.4.tar.gz openbsd-libressl-v2.7.4.tar.bz2 openbsd-libressl-v2.7.4.zip | |
MFC: Reject excessively large primes in DH key generation. Problem reportedlibressl-v2.7.4
by Guido Vranken to OpenSSL (https://github.com/openssl/openssl/pull/6457)
and based on his diff. suggestions from tb@, ok tb@ jsing@
Original commit by sthen@
Diffstat (limited to 'src/lib')
| -rw-r--r-- | src/lib/libcrypto/dh/dh_key.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/lib/libcrypto/dh/dh_key.c b/src/lib/libcrypto/dh/dh_key.c index 63d38771c3..d79e98e785 100644 --- a/src/lib/libcrypto/dh/dh_key.c +++ b/src/lib/libcrypto/dh/dh_key.c | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: dh_key.c,v 1.27 2017/01/29 17:49:22 beck Exp $ */ | 1 | /* $OpenBSD: dh_key.c,v 1.27.2.1 2018/06/13 15:13:30 jsing Exp $ */ |
| 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) | 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * | 4 | * |
| @@ -108,6 +108,11 @@ generate_key(DH *dh) | |||
| 108 | BN_MONT_CTX *mont = NULL; | 108 | BN_MONT_CTX *mont = NULL; |
| 109 | BIGNUM *pub_key = NULL, *priv_key = NULL; | 109 | BIGNUM *pub_key = NULL, *priv_key = NULL; |
| 110 | 110 | ||
| 111 | if (BN_num_bits(dh->p) > OPENSSL_DH_MAX_MODULUS_BITS) { | ||
| 112 | DHerror(DH_R_MODULUS_TOO_LARGE); | ||
| 113 | return 0; | ||
| 114 | } | ||
| 115 | |||
| 111 | ctx = BN_CTX_new(); | 116 | ctx = BN_CTX_new(); |
| 112 | if (ctx == NULL) | 117 | if (ctx == NULL) |
| 113 | goto err; | 118 | goto err; |
