diff options
| author | Adam Langley <agl@google.com> | 2015-04-13 21:04:14 +0300 |
|---|---|---|
| committer | Adam Langley <agl@google.com> | 2015-04-14 23:10:27 +0300 |
| commit | 683d7bd20a96a34d85341cd04b4c6309b0730852 (patch) | |
| tree | e6f9f01d41862b133f97936b973f7e4e825bc802 /include/openssl/rsa.h | |
| parent | ccdfbd9834511a473684caf4517bc5dcdf00fc58 (diff) | |
Convert BN_MONT_CTX to new-style locking.
This introduces a per-RSA/DSA/DH lock. This is good for lock contention,
although pthread locks are depressingly bloated.
Change-Id: I07c4d1606fc35135fc141ebe6ba904a28c8f8a0c
Reviewed-on: https://boringssl-review.googlesource.com/4324
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'include/openssl/rsa.h')
| -rw-r--r-- | include/openssl/rsa.h | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/include/openssl/rsa.h b/include/openssl/rsa.h index f49eb143..889ad192 100644 --- a/include/openssl/rsa.h +++ b/include/openssl/rsa.h @@ -61,6 +61,7 @@ #include <openssl/engine.h> #include <openssl/ex_data.h> +#include <openssl/thread.h> #if defined(__cplusplus) extern "C" { @@ -471,18 +472,21 @@ struct rsa_st { int references; int flags; - /* Used to cache montgomery values */ + CRYPTO_MUTEX lock; + + /* Used to cache montgomery values. The creation of these values is protected + * by |lock|. */ BN_MONT_CTX *_method_mod_n; BN_MONT_CTX *_method_mod_p; BN_MONT_CTX *_method_mod_q; /* num_blindings contains the size of the |blindings| and |blindings_inuse| * arrays. This member and the |blindings_inuse| array are protected by - * CRYPTO_LOCK_RSA_BLINDING. */ + * |lock|. */ unsigned num_blindings; /* blindings is an array of BN_BLINDING structures that can be reserved by a - * thread by locking CRYPTO_LOCK_RSA_BLINDING and changing the corresponding - * element in |blindings_inuse| from 0 to 1. */ + * thread by locking |lock| and changing the corresponding element in + * |blindings_inuse| from 0 to 1. */ BN_BLINDING **blindings; unsigned char *blindings_inuse; }; |