diff options
| author | Geoff Thorpe <geoff@openssl.org> | 2008-08-06 17:54:15 +0200 |
|---|---|---|
| committer | Geoff Thorpe <geoff@openssl.org> | 2008-08-06 17:54:15 +0200 |
| commit | 4c3296960de32e5abfbb8f4703a2ce624d82669f (patch) | |
| tree | f603740bcbf0e3cde6008cead9586e5b4edbdc5b /crypto/rsa/rsa_eay.c | |
| parent | sha1-armv4-large cosmetics. (diff) | |
| download | openssl-4c3296960de32e5abfbb8f4703a2ce624d82669f.tar.xz openssl-4c3296960de32e5abfbb8f4703a2ce624d82669f.zip | |
Remove the dual-callback scheme for numeric and pointer thread IDs,
deprecate the original (numeric-only) scheme, and replace with the
CRYPTO_THREADID object. This hides the platform-specifics and should reduce
the possibility for programming errors (where failing to explicitly check
both thread ID forms could create subtle, platform-specific bugs).
Thanks to Bodo, for invaluable review and feedback.
Diffstat (limited to 'crypto/rsa/rsa_eay.c')
| -rw-r--r-- | crypto/rsa/rsa_eay.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/crypto/rsa/rsa_eay.c b/crypto/rsa/rsa_eay.c index a1ecd6d478..7321349237 100644 --- a/crypto/rsa/rsa_eay.c +++ b/crypto/rsa/rsa_eay.c @@ -264,6 +264,7 @@ static BN_BLINDING *rsa_get_blinding(RSA *rsa, int *local, BN_CTX *ctx) { BN_BLINDING *ret; int got_write_lock = 0; + CRYPTO_THREADID cur; CRYPTO_r_lock(CRYPTO_LOCK_RSA); @@ -281,7 +282,8 @@ static BN_BLINDING *rsa_get_blinding(RSA *rsa, int *local, BN_CTX *ctx) if (ret == NULL) goto err; - if ((BN_BLINDING_get_thread_id(ret) == CRYPTO_thread_id()) && (BN_BLINDING_get_thread_idptr(ret) == CRYPTO_thread_idptr())) + CRYPTO_THREADID_current(&cur); + if (!CRYPTO_THREADID_cmp(&cur, BN_BLINDING_thread_id(ret))) { /* rsa->blinding is ours! */ |