On Mon, Dec 16, 2002, Richard Levitte wrote:

>     Protect loading routines with a lock.
>     PR: 373
>
>   Summary:
>     Revision    Changes     Path
>     1.33.2.4    +6  -1      openssl/ssl/ssl_ciph.c
>   ____________________________________________________________________________
>
>   Index: openssl/ssl/ssl_ciph.c
>   ============================================================
>   $ cvs diff -u -r1.33.2.3 -r1.33.2.4 ssl_ciph.c
>   --- openssl/ssl/ssl_ciph.c  19 Jul 2002 19:53:02 -0000      1.33.2.3
>   +++ openssl/ssl/ssl_ciph.c  16 Dec 2002 06:06:06 -0000      1.33.2.4
>   @@ -751,7 +751,12 @@
>        */
>       if (rule_str == NULL) return(NULL);
>
>   -   if (init_ciphers) load_ciphers();
>   +   if (init_ciphers)
>   +           {
>   +           CRYPTO_w_lock(CRYPTO_LOCK_SSL);
>   +           if (init_ciphers) load_ciphers();
>   +           CRYPTO_w_unlock(CRYPTO_LOCK_SSL);
>   +           }

Just curious: why the nested "if (init_ciphers) .."?

                                       Ralf S. Engelschall
                                       [EMAIL PROTECTED]
                                       www.engelschall.com
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to