Tell openssl how to use locks and how to find thread ids -- this may prevent race conditions surrounding the error queue.

svn:r3622
This commit is contained in:
Nick Mathewson 2005-02-13 22:32:25 +00:00
parent 427e246a10
commit aac97a3c6a
3 changed files with 44 additions and 0 deletions

View File

@ -72,6 +72,10 @@ const char compat_c_id[] = "$Id$";
#include "log.h" #include "log.h"
#include "util.h" #include "util.h"
#ifdef TOR_IS_MULTITHREADED
#include <openssl/crypto.h>
#endif
/* Inline the strl functions if the platform doesn't have them. */ /* Inline the strl functions if the platform doesn't have them. */
#ifndef HAVE_STRLCPY #ifndef HAVE_STRLCPY
#include "strlcpy.c" #include "strlcpy.c"
@ -774,6 +778,11 @@ void tor_mutex_release(tor_mutex_t *m)
log_fn(LOG_WARN, "Failed to release mutex: %d", GetLastError()); log_fn(LOG_WARN, "Failed to release mutex: %d", GetLastError());
} }
} }
unsigned long
tor_get_thread_id(void)
{
return (unsigned long)GetCurrentThreadId();
}
#elif defined(USE_PTHREADS) #elif defined(USE_PTHREADS)
struct tor_mutex_t { struct tor_mutex_t {
pthread_mutex_t mutex; pthread_mutex_t mutex;
@ -800,6 +809,11 @@ void tor_mutex_free(tor_mutex_t *m)
pthread_mutex_destroy(&m->mutex); pthread_mutex_destroy(&m->mutex);
tor_free(m); tor_free(m);
} }
unsigned long
tor_get_thread_id(void)
{
return (unsigned long)pthread_self();
}
#else #else
struct tor_mutex_t { struct tor_mutex_t {
int _unused; int _unused;
@ -808,6 +822,7 @@ tor_mutex_t *tor_mutex_new(void) { return NULL; }
void tor_mutex_acquire(tor_mutex_t *m) { } void tor_mutex_acquire(tor_mutex_t *m) { }
void tor_mutex_release(tor_mutex_t *m) { } void tor_mutex_release(tor_mutex_t *m) { }
void tor_mutex_free(tor_mutex_t *m) { } void tor_mutex_free(tor_mutex_t *m) { }
unsigned long tor_get_thread_id(void) { return 1; }
#endif #endif
/** /**

View File

@ -199,6 +199,7 @@ tor_mutex_t *tor_mutex_new(void);
void tor_mutex_acquire(tor_mutex_t *m); void tor_mutex_acquire(tor_mutex_t *m);
void tor_mutex_release(tor_mutex_t *m); void tor_mutex_release(tor_mutex_t *m);
void tor_mutex_free(tor_mutex_t *m); void tor_mutex_free(tor_mutex_t *m);
unsigned long tor_get_thread_id(void);
#if defined(MS_WINDOWS) #if defined(MS_WINDOWS)
#define USE_WIN32_THREADS #define USE_WIN32_THREADS

View File

@ -55,6 +55,7 @@ const char crypto_c_id[] = "$Id$";
#include "aes.h" #include "aes.h"
#include "util.h" #include "util.h"
#include "container.h" #include "container.h"
#include "compat.h"
#if OPENSSL_VERSION_NUMBER < 0x00905000l #if OPENSSL_VERSION_NUMBER < 0x00905000l
#error "We require openssl >= 0.9.5" #error "We require openssl >= 0.9.5"
@ -100,6 +101,8 @@ RSA *_crypto_pk_env_get_rsa(crypto_pk_env_t *env);
EVP_PKEY *_crypto_pk_env_get_evp_pkey(crypto_pk_env_t *env, int private); EVP_PKEY *_crypto_pk_env_get_evp_pkey(crypto_pk_env_t *env, int private);
DH *_crypto_dh_env_get_dh(crypto_dh_env_t *dh); DH *_crypto_dh_env_get_dh(crypto_dh_env_t *dh);
static int setup_openssl_threading(void);
/** Return the number of bytes added by padding method <b>padding</b>. /** Return the number of bytes added by padding method <b>padding</b>.
*/ */
static INLINE int static INLINE int
@ -157,6 +160,7 @@ int crypto_global_init()
if (!_crypto_global_initialized) { if (!_crypto_global_initialized) {
ERR_load_crypto_strings(); ERR_load_crypto_strings();
_crypto_global_initialized = 1; _crypto_global_initialized = 1;
setup_openssl_threading();
} }
return 0; return 0;
} }
@ -1626,3 +1630,27 @@ secret_to_key(char *key_out, size_t key_out_len, const char *secret,
crypto_free_digest_env(d); crypto_free_digest_env(d);
} }
#ifdef TOR_IS_MULTITHREADED
static tor_mutex_t **_openssl_mutexes = NULL;
static void
_openssl_locking_cb(int mode, int n, const char *file, int line)
{
if (mode & CRYPTO_LOCK)
tor_mutex_acquire(_openssl_mutexes[n]);
else
tor_mutex_release(_openssl_mutexes[n]);
}
static int
setup_openssl_threading(void) {
int i;
int n = CRYPTO_num_locks();
_openssl_mutexes = tor_malloc(n*sizeof(tor_mutex_t *));
for (i=0; i <n; ++i)
_openssl_mutexes[i] = tor_mutex_new();
CRYPTO_set_locking_callback(_openssl_locking_cb);
CRYPTO_set_id_callback(tor_get_thread_id);
return 0;
}
#else
static int setup_openssl_threading(void) { return 0; }
#endif