On Tue, Sep 24, 2002 at 03:47:14PM -0700, Patrick McCormick wrote:

> Many thanks for putting in a lock.  However, the race condition has not been
> eliminated.
> [...]            init must be checked after the lock is entered in order to
> prevent the client_data setup from happening twice.  So, add:
> 
>     if (init)
>         {
>         CRYPTO_w_lock(CRYPTO_LOCK_SSL_METHOD);
>         if (init)
>             {
> ....
>             }
>         CRYPTO_w_unlock(CRYPTO_LOCK_SSL_METHOD);
>         }

You are absolutely right, of course.  I got similar constructs right
in the past, but I guess this time I was too busy with various other
things to really think about what I was writing ...

The next snapshot should really fix the problem.


-- 
Bodo M�ller <[EMAIL PROTECTED]>
PGP http://www.informatik.tu-darmstadt.de/TI/Mitarbeiter/moeller/0x36d2c658.html
* TU Darmstadt, Theoretische Informatik, Alexanderstr. 10, D-64283 Darmstadt
* Tel. +49-6151-16-6628, Fax +49-6151-16-6036

______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to