diff --git a/crypto/threads_none.c b/crypto/threads_none.c index ab6db8e36b988519e42bb3f74829b144c6cc405b..4b1940ae44dbd63a50df824a5d876633a2786796 100644 --- a/crypto/threads_none.c +++ b/crypto/threads_none.c @@ -128,18 +128,6 @@ int CRYPTO_atomic_add(int *val, int amount, int *ret, CRYPTO_RWLOCK *lock) return 1; } -int CRYPTO_atomic_read(int *val, int *ret, CRYPTO_RWLOCK *lock) -{ - *ret = *val; - return 1; -} - -int CRYPTO_atomic_write(int *val, int n, CRYPTO_RWLOCK *lock) -{ - *val = n; - return 1; -} - int openssl_init_fork_handlers(void) { return 0; diff --git a/crypto/threads_pthread.c b/crypto/threads_pthread.c index 8d30e1c69f3587ca6b200f3f2823eef01ede6f64..5a59779ebbb1d9b15267c35b74e907d467b12d6c 100644 --- a/crypto/threads_pthread.c +++ b/crypto/threads_pthread.c @@ -175,44 +175,6 @@ int CRYPTO_atomic_add(int *val, int amount, int *ret, CRYPTO_RWLOCK *lock) return 1; } -int CRYPTO_atomic_read(int *val, int *ret, CRYPTO_RWLOCK *lock) -{ -# if defined(__GNUC__) && defined(__ATOMIC_ACQUIRE) - if (__atomic_is_lock_free(sizeof(*val), val)) { - __atomic_load(val, ret, __ATOMIC_ACQUIRE); - return 1; - } -# endif - if (!CRYPTO_THREAD_read_lock(lock)) - return 0; - - *ret = *val; - - if (!CRYPTO_THREAD_unlock(lock)) - return 0; - - return 1; -} - -int CRYPTO_atomic_write(int *val, int n, CRYPTO_RWLOCK *lock) -{ -# if defined(__GNUC__) && defined(__ATOMIC_RELEASE) - if (__atomic_is_lock_free(sizeof(*val), val)) { - __atomic_store(val, &n, __ATOMIC_RELEASE); - return 1; - } -# endif - if (!CRYPTO_THREAD_write_lock(lock)) - return 0; - - *val = n; - - if (!CRYPTO_THREAD_unlock(lock)) - return 0; - - return 1; -} - # ifdef OPENSSL_SYS_UNIX static pthread_once_t fork_once_control = PTHREAD_ONCE_INIT; diff --git a/crypto/threads_win.c b/crypto/threads_win.c index 7fdbc1f67fe5505c9f8eaf2563dc18a8837215e9..d8fdfb74f5b7b093ed3b926eb005c10c4f3df681 100644 --- a/crypto/threads_win.c +++ b/crypto/threads_win.c @@ -155,18 +155,6 @@ int CRYPTO_atomic_add(int *val, int amount, int *ret, CRYPTO_RWLOCK *lock) return 1; } -int CRYPTO_atomic_read(int *val, int *ret, CRYPTO_RWLOCK *lock) -{ - *ret = InterlockedCompareExchange(val, 0, 0); - return 1; -} - -int CRYPTO_atomic_write(int *val, int n, CRYPTO_RWLOCK *lock) -{ - InterlockedExchange(val, n); - return 1; -} - int openssl_init_fork_handlers(void) { return 0; diff --git a/doc/man3/CRYPTO_THREAD_run_once.pod b/doc/man3/CRYPTO_THREAD_run_once.pod index a290278b70d57e0b544184edd53460d5c100b5fb..32776131936c8a7a8000740b83dd59cfcd6fcb97 100644 --- a/doc/man3/CRYPTO_THREAD_run_once.pod +++ b/doc/man3/CRYPTO_THREAD_run_once.pod @@ -4,8 +4,8 @@ CRYPTO_THREAD_run_once, CRYPTO_THREAD_lock_new, CRYPTO_THREAD_read_lock, CRYPTO_THREAD_write_lock, -CRYPTO_THREAD_unlock, CRYPTO_THREAD_lock_free, CRYPTO_atomic_add, -CRYPTO_atomic_read, CRYPTO_atomic_write - OpenSSL thread support +CRYPTO_THREAD_unlock, CRYPTO_THREAD_lock_free, +CRYPTO_atomic_add - OpenSSL thread support =head1 SYNOPSIS @@ -21,8 +21,6 @@ CRYPTO_atomic_read, CRYPTO_atomic_write - OpenSSL thread support void CRYPTO_THREAD_lock_free(CRYPTO_RWLOCK *lock); int CRYPTO_atomic_add(int *val, int amount, int *ret, CRYPTO_RWLOCK *lock); - int CRYPTO_atomic_read(int *val, int *ret, CRYPTO_RWLOCK *lock); - int CRYPTO_atomic_write(int *val, int n, CRYPTO_RWLOCK *lock); =head1 DESCRIPTION @@ -77,17 +75,6 @@ operations are supported on the specific platform. Because of this, if a variable is modified by CRYPTO_atomic_add() then CRYPTO_atomic_add() must be the only way that the variable is modified. -=item * - -CRYPTO_atomic_read() atomically reads B and returns the result of -the operation in B. B will be locked, unless atomic operations -are supported on the specific platform. - -=item * - -CRYPTO_atomic_write() atomically writes B to B. B will be -locked, unless atomic operations are supported on the specific platform. - =back =head1 RETURN VALUES diff --git a/include/openssl/crypto.h b/include/openssl/crypto.h index c2ad65e044a04b47ee392b6c217166d75c88cf84..7e50b1bf4694921b5da4845a04faf5674ae7ee7e 100644 --- a/include/openssl/crypto.h +++ b/include/openssl/crypto.h @@ -73,8 +73,6 @@ int CRYPTO_THREAD_unlock(CRYPTO_RWLOCK *lock); void CRYPTO_THREAD_lock_free(CRYPTO_RWLOCK *lock); int CRYPTO_atomic_add(int *val, int amount, int *ret, CRYPTO_RWLOCK *lock); -int CRYPTO_atomic_read(int *val, int *ret, CRYPTO_RWLOCK *lock); -int CRYPTO_atomic_write(int *val, int n, CRYPTO_RWLOCK *lock); /* * The following can be used to detect memory leaks in the library. If diff --git a/util/libcrypto.num b/util/libcrypto.num index ed8a8f0127780c2add82c40fc090be9af277e081..d69a6dcf688ea19c93aa9c21aec213ccceed54aa 100644 --- a/util/libcrypto.num +++ b/util/libcrypto.num @@ -4396,8 +4396,6 @@ EVP_PKEY_meth_set_check 4341 1_1_1 EXIST::FUNCTION: EVP_PKEY_meth_get_check 4342 1_1_1 EXIST::FUNCTION: EVP_PKEY_meth_remove 4343 1_1_1 EXIST::FUNCTION: OPENSSL_sk_reserve 4344 1_1_1 EXIST::FUNCTION: -CRYPTO_atomic_read 4345 1_1_1 EXIST::FUNCTION: -CRYPTO_atomic_write 4346 1_1_1 EXIST::FUNCTION: EVP_PKEY_set1_engine 4347 1_1_0g EXIST::FUNCTION:ENGINE DH_new_by_nid 4348 1_1_1 EXIST::FUNCTION:DH DH_get_nid 4349 1_1_1 EXIST::FUNCTION:DH