Revert "Fixed deadlock in CRYPTO_THREAD_run_once for Windows"
authorMatt Caswell <matt@openssl.org>
Mon, 14 Nov 2016 12:00:45 +0000 (12:00 +0000)
committerMatt Caswell <matt@openssl.org>
Mon, 14 Nov 2016 12:00:45 +0000 (12:00 +0000)
commitc028052c4cfc39dc99f735f1743b18867547129f
treec1707605f7ab5ba360b5234a2f8211da92b3c6c5
parentedc18749bd5dfb7e12513d3978f78f9b56104fd6
Revert "Fixed deadlock in CRYPTO_THREAD_run_once for Windows"

This reverts commit edc18749bd5dfb7e12513d3978f78f9b56104fd6.

The proposed fix is incorrect. It marks the "run_once" code as having
finished before it has. The intended semantics of run_once is that no
threads should proceed until the code has run exactly once. With this
change the "second" thread will think the run_once code has already been
run and will continue, even though it is still in progress. This could
result in a crash or other incorrect behaviour.

Reviewed-by: Tim Hudson <tjh@openssl.org>
crypto/threads_win.c