diff options
author | slontis <shane.lontis@oracle.com> | 2023-02-27 13:48:24 +1000 |
---|---|---|
committer | Pauli <pauli@openssl.org> | 2023-03-01 09:20:49 +1100 |
commit | 3a4e09ab42654b3d223f0f8dd1a9c58b2902ddcc (patch) | |
tree | ef08d17eec3bef6ddbf411f5876c9e9d1e8ddcab /include/openssl | |
parent | a6d7093a1dc890493d3577c738e729c0265d7b40 (diff) | |
download | openssl-new-3a4e09ab42654b3d223f0f8dd1a9c58b2902ddcc.tar.gz |
Fix infinite loops in DSA sign code.
Fixes #20268
Values such as q=1 or priv=0 caused infinite loops when calling
DSA_sign() without these changes.
There are other cases where bad domain parameters may have caused
infinite loops where the retry counter has been added. The simpler case
of priv=0 also hits this case. q=1 caused an infinite loop in the setup.
The max retry value has been set to an arbitrary value of 8 (it is
unlikely to ever do a single retry for valid values).
The minimum q bits was set to an arbitrary value of 128 (160 is still
used for legacy reasons when using 512 bit keys).
Thanks @guidovranken for detecting this, and @davidben for his
insightful analysis.
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/20384)
Diffstat (limited to 'include/openssl')
-rw-r--r-- | include/openssl/dsaerr.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/include/openssl/dsaerr.h b/include/openssl/dsaerr.h index 5f0ca8d12a..26ada57d80 100644 --- a/include/openssl/dsaerr.h +++ b/include/openssl/dsaerr.h @@ -1,6 +1,6 @@ /* * Generated by util/mkerr.pl DO NOT EDIT - * Copyright 1995-2021 The OpenSSL Project Authors. All Rights Reserved. + * Copyright 1995-2023 The OpenSSL Project Authors. All Rights Reserved. * * Licensed under the Apache License 2.0 (the "License"). You may not use * this file except in compliance with the License. You can obtain a copy @@ -38,6 +38,7 @@ # define DSA_R_P_NOT_PRIME 115 # define DSA_R_Q_NOT_PRIME 113 # define DSA_R_SEED_LEN_SMALL 110 +# define DSA_R_TOO_MANY_RETRIES 116 # endif #endif |