crypto: replace rwlocks with simple mutexes
authorBen Noordhuis <info@bnoordhuis.nl>
Mon, 7 Sep 2015 12:32:00 +0000 (14:32 +0200)
committerRod Vagg <rod@vagg.org>
Tue, 8 Sep 2015 00:35:14 +0000 (10:35 +1000)
commit03f900ab25b951800783a1a7a583280006fb1731
treec20f49ba2e39a8c92aa7fa30f9ad48c436b15824
parent81a8d451f9a2ca71ece66f61785a1bd939c8c665
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: https://github.com/nodejs/node/pull/2723
Reviewed-By: Fedor Indutny <fedor@indutny.com>
src/node_crypto.cc