Commit 03f900a
crypto: replace rwlocks with simple mutexes
It was pointed out by Zhou Ran that the Windows XP implementation of
uv_rwlock_rdlock() and friends may unlock the inner write mutex on a
different thread than the one that locked it, resulting in undefined
behavior.
The only place that uses rwlocks is the crypto module. Make that use
normal (simple) mutexes instead.
OpenSSL's critical sections are generally very short, with exclusive
access outnumbering shared access by a factor of three or more, so
it's not as if using rwlocks gives a decisive performance advantage.
PR-URL: #2723
Reviewed-By: Fedor Indutny <fedor@indutny.com>1 parent 81a8d45 commit 03f900a
1 file changed
Lines changed: 9 additions & 16 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -123,7 +123,7 @@ struct ClearErrorOnReturn { | |||
| 123 | 123 | ~ClearErrorOnReturn() { ERR_clear_error(); } | |
| 124 | 124 | }; | |
| 125 | 125 | ||
| 126 | - static uv_rwlock_t* locks; | ||
| 126 | + static uv_mutex_t* locks; | ||
| 127 | 127 | ||
| 128 | 128 | const char* const root_certs[] = { | |
| 129 | 129 | #include "node_root_certs.h" // NOLINT(build/include_order) | |
@@ -178,29 +178,22 @@ static void crypto_lock_init(void) { | |||
| 178 | 178 | int i, n; | |
| 179 | 179 | ||
| 180 | 180 | n = CRYPTO_num_locks(); | |
| 181 | - locks = new uv_rwlock_t[n]; | ||
| 181 | + locks = new uv_mutex_t[n]; | ||
| 182 | 182 | ||
| 183 | 183 | for (i = 0; i < n; i++) | |
| 184 | - if (uv_rwlock_init(locks + i)) | ||
| 184 | + if (uv_mutex_init(locks + i)) | ||
| 185 | 185 | abort(); | |
| 186 | 186 | } | |
| 187 | 187 | ||
| 188 | 188 | ||
| 189 | 189 | static void crypto_lock_cb(int mode, int n, const char* file, int line) { | |
| 190 | - CHECK((mode & CRYPTO_LOCK) || (mode & CRYPTO_UNLOCK)); | ||
| 191 | - CHECK((mode & CRYPTO_READ) || (mode & CRYPTO_WRITE)); | ||
| 190 | + CHECK(!(mode & CRYPTO_LOCK) ^ !(mode & CRYPTO_UNLOCK)); | ||
| 191 | + CHECK(!(mode & CRYPTO_READ) ^ !(mode & CRYPTO_WRITE)); | ||
| 192 | 192 | ||
| 193 | - if (mode & CRYPTO_LOCK) { | ||
| 194 | - if (mode & CRYPTO_READ) | ||
| 195 | - uv_rwlock_rdlock(locks + n); | ||
| 196 | - else | ||
| 197 | - uv_rwlock_wrlock(locks + n); | ||
| 198 | - } else { | ||
| 199 | - if (mode & CRYPTO_READ) | ||
| 200 | - uv_rwlock_rdunlock(locks + n); | ||
| 201 | - else | ||
| 202 | - uv_rwlock_wrunlock(locks + n); | ||
| 203 | - } | ||
| 193 | + if (mode & CRYPTO_LOCK) | ||
| 194 | + uv_mutex_lock(locks + n); | ||
| 195 | + else | ||
| 196 | + uv_mutex_unlock(locks + n); | ||
| 204 | 197 | } | |
| 205 | 198 | ||
| 206 | 199 | ||
0 commit comments