#18295 [Asn->Csd]: failure compile with --openssl : e_os.h don't exist in OpenSSL-0.9.7b

ID: 18295 Updated by: kalowsky@php.net Reported By: mmacwing@irisa.fr -Status: Assigned +Status: Closed Bug Type: Compile Failure Operating System: Solaris 2.7 PHP Version: 4.2.1 Assigned To: sniper New Comment: Message from Ricahrd Levitte (OpenSSL Developer): The openssl application does that by concatenating the result from X509_get_default_cert_area() with the string "/openssl.cnf". So what we have in CVS is correct. Marking as closed. Previous Comments: ------------------------------------------------------------------------ [2002-07-29 13:29:54] xavier@nextdimensioninc.com Hi there, I tried to email this message to sanders@php.net thinking it would be more appropriate, unfortunately, the "Self-moderation" system asked me to confirm my email address by replying or sending a response to an email address returned as non-existent by your MDA, you should do something about that :( Anyway, concerning this bug, I believe you will most certainly find insight by going through the source code of mod_ssl 2.8-10 (http://www.modssl.org I believe). This should help you find all the directives you require to find a final solution. Thanks. Xavier Spriet. Developer. xavier@nextdimensioninc.com ------------------------------------------------------------------------ [2002-07-13 14:55:21] sander@php.net My patch actually wasn't the right way to fix it. (see http://marc.theaimsgroup.com/?l=php-cvs&m=102658634714044&w=2) Reopening... ------------------------------------------------------------------------ [2002-07-12 17:46:40] sniper@php.net This bug has been fixed in CVS. You can grab a snapshot of the CVS version at http://snaps.php.net/. In case this was a documentation problem, the fix will show up soon at http://www.php.net/manual/. In case this was a PHP.net website problem, the change will show up on the PHP.net site and on the mirror sites. Thank you for the report, and for helping us make PHP better. ------------------------------------------------------------------------ [2002-07-12 15:48:45] mmacwing@irisa.fr Thanks and excuse me for the others duplicated messages bugs... Ok I will try this patch this monday. Michel. ------------------------------------------------------------------------ [2002-07-12 15:12:17] sander@php.net The patch below fixes the problem (against CVS HEAD, but should work on 4.2.1 too), but I'm not sure if this is the right way to fix it. OPENSSL_CONF is only defined in e_os.h, so I just grabbed it's value. Any hints/ideas are appreciated. Index: openssl.c =================================================================== RCS file: /repository/php4/ext/openssl/openssl.c,v retrieving revision 1.49 diff -u -r1.49 openssl.c --- openssl.c 13 Jun 2002 19:41:21 -0000 1.49 +++ openssl.c 12 Jul 2002 19:10:14 -0000 @@ -39,7 +39,6 @@ #include <openssl/pem.h> #include <openssl/err.h> #include <openssl/conf.h> -#include <openssl/e_os.h> #include <openssl/rand.h> #define DEFAULT_KEY_LENGTH 512 @@ -579,7 +578,7 @@ if (config_filename == NULL) { snprintf(default_ssl_conf_filename, sizeof(default_ssl_conf_filename), "%s/%s", X509_get_default_cert_area(), - OPENSSL_CONF); + "openssl.cnf"); } else strncpy(default_ssl_conf_filename, config_filename, sizeof(default_ssl_conf_filename)); ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/18295 -- Edit this bug report at http://bugs.php.net/?id=18295&edit=1