diff options
author | David Benjamin <davidben@google.com> | 2022-10-04 15:36:28 -0400 |
---|---|---|
committer | Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-10-04 22:11:36 +0000 |
commit | 5a9043a0ff3a5d952a8cc4c8dd24650dd84cf64e (patch) | |
tree | 5a93fb81f22322b14d347cbf4209495eac11d6ed /crypto/dh_extra/dh_asn1.c | |
parent | 64393b57e8734b92a6ba784bcfc02b1aa01e5ff2 (diff) | |
download | boringssl-5a9043a0ff3a5d952a8cc4c8dd24650dd84cf64e.zip boringssl-5a9043a0ff3a5d952a8cc4c8dd24650dd84cf64e.tar.gz boringssl-5a9043a0ff3a5d952a8cc4c8dd24650dd84cf64e.tar.bz2 |
Make DH opaque.
In doing so, remove some X9.42 placeholder fields, since it's impossible
to set them. I switched dh_test.cc to the getters where it was easy, but
OpenSSL's new setters are so tedious that I just gave it access to the
internal struct.
With this, there are now only two public structs (DSA and RSA) that
reference CRYPTO_MUTEX. After that's removed, we can stop worrying about
pthread_rwlock_t feature flags in the public headers.
Update-Note: DH is now an opaque structure. Callers should use accessors
instead of accessing fields.
Change-Id: Ia53702f8ab58884a90d85718ee26eb03d062d234
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/54625
Reviewed-by: Bob Beck <bbe@google.com>
Auto-Submit: David Benjamin <davidben@google.com>
Commit-Queue: David Benjamin <davidben@google.com>
Diffstat (limited to 'crypto/dh_extra/dh_asn1.c')
-rw-r--r-- | crypto/dh_extra/dh_asn1.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/crypto/dh_extra/dh_asn1.c b/crypto/dh_extra/dh_asn1.c index 9d32180..de01077 100644 --- a/crypto/dh_extra/dh_asn1.c +++ b/crypto/dh_extra/dh_asn1.c @@ -63,6 +63,7 @@ #include <openssl/err.h> #include "../bytestring/internal.h" +#include "../fipsmodule/dh/internal.h" static int parse_integer(CBS *cbs, BIGNUM **out) { |