Re: cvs commit: httpd-2.0/modules/ssl mod_ssl.c

[Erik Abele <er...@codefaktor.de>]

Hmmm,

but apps/apps.h and some example apps (rsautl.c, pkcs8.c) are using 
OpenSSL_add_all_algorithms() instead of SSL_load_library() and 
furthermore it seems that the complete source tree (0.9.6 & 0.9.7 CVS 
HEAD of yesterday) lacks any reference to SSL_load_library()?!

Perhaps I'm blind but building 2.1 together with mod_ssl and OpenSSL 
0.9.7b is currently broken for me:

modules/ssl/.libs/mod_ssl.al(mod_ssl.lo)(.text+0x26): In function 
`ssl_hook_pre_config':
/tmp/httpd-install/modules/ssl/mod_ssl.c:243: undefined reference to 
`SSL_load_library'

Cheers,
Erik

wrowe@apache.org wrote:
> wrowe       2003/05/30 14:35:55
> 
>   Modified:    modules/ssl mod_ssl.c
>   Log:
>     OpenSSL_add_all_algorithms is simply an alias for SSL_load_library.
>   
>     Note that the entire schema of what-we-load-how follows from
>     OpenSSL 0.9.7's own apps/ example applications.  More review
>     is greatly desired, but that's where I believed I should
>     start looking for the 'correct' order of operations.
>   
>   Revision  Changes    Path
>   1.81      +1 -1      httpd-2.0/modules/ssl/mod_ssl.c
>   
>   Index: mod_ssl.c
>   ===================================================================
>   RCS file: /home/cvs/httpd-2.0/modules/ssl/mod_ssl.c,v
>   retrieving revision 1.80
>   retrieving revision 1.81
>   diff -u -r1.80 -r1.81
>   --- mod_ssl.c	30 May 2003 20:58:55 -0000	1.80
>   +++ mod_ssl.c	30 May 2003 21:35:04 -0000	1.81
>   @@ -240,7 +240,7 @@
>         */
>        CRYPTO_malloc_init();
>        ERR_load_crypto_strings();
>   -    OpenSSL_add_all_algorithms();
>   +    SSL_load_library();
>    #if HAVE_ENGINE_LOAD_BUILTIN_ENGINES
>        ENGINE_load_builtin_engines();
>    #endif
>