You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by jo...@apache.org on 2020/02/14 08:53:22 UTC

svn commit: r1874005 - /httpd/httpd/trunk/modules/ssl/ssl_engine_init.c

Author: jorton
Date: Fri Feb 14 08:53:21 2020
New Revision: 1874005

URL: http://svn.apache.org/viewvc?rev=1874005&view=rev
Log:
* modules/ssl/ssl_engine_init.c (ssl_init_Module): Avoid some bogus
  gcc -Wmaybe-uninitialized warnings in (slightly odd) SSLFIPS
  handling.

Modified:
    httpd/httpd/trunk/modules/ssl/ssl_engine_init.c

Modified: httpd/httpd/trunk/modules/ssl/ssl_engine_init.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/ssl/ssl_engine_init.c?rev=1874005&r1=1874004&r2=1874005&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/ssl/ssl_engine_init.c (original)
+++ httpd/httpd/trunk/modules/ssl/ssl_engine_init.c Fri Feb 14 08:53:21 2020
@@ -331,23 +331,26 @@ apr_status_t ssl_init_Module(apr_pool_t
     ssl_rand_seed(base_server, ptemp, SSL_RSCTX_STARTUP, "Init: ");
 
 #ifdef HAVE_FIPS
+    /* ### The FIPS setting is global and must be the same in all
+     * SSLSrvConfigRecs, should be in SSLModConfigRec really. */
+    sc = mySrvConfig(base_server);
     if (sc->fips) {
         if (!FIPS_mode()) {
             if (FIPS_mode_set(1)) {
-                ap_log_error(APLOG_MARK, APLOG_NOTICE, 0, s, APLOGNO(01884)
+                ap_log_error(APLOG_MARK, APLOG_NOTICE, 0, base_server, APLOGNO(01884)
                              "Operating in SSL FIPS mode");
                 apr_pool_cleanup_register(p, NULL, modssl_fips_cleanup,
                                           apr_pool_cleanup_null);
             }
             else {
-                ap_log_error(APLOG_MARK, APLOG_EMERG, 0, s, APLOGNO(01885) "FIPS mode failed");
-                ssl_log_ssl_error(SSLLOG_MARK, APLOG_EMERG, s);
-                return ssl_die(s);
+                ap_log_error(APLOG_MARK, APLOG_EMERG, 0, base_server, APLOGNO(01885) "FIPS mode failed");
+                ssl_log_ssl_error(SSLLOG_MARK, APLOG_EMERG, base_server);
+                return ssl_die(base_server);
             }
         }
     }
     else {
-        ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, s, APLOGNO(01886)
+        ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, base_server, APLOGNO(01886)
                      "SSL FIPS mode disabled");
     }
 #endif