Re: svn commit: r768504 - in /httpd/httpd/branches/2.2.x: CHANGES STATUS docs/manual/mod/mod_ssl.xml modules/proxy/mod_proxy_http.c modules/ssl/mod_ssl.c modules/ssl/ssl_engine_config.c modules/ssl/ssl_engine_io.c modules/ssl/ssl_private.h

[Roy T. Fielding]

On May 4, 2009, at 12:22 PM, Ruediger Pluem wrote:

During my work on the SNI backport I noticed that during my preparation of the backport patch for this proposal one import line fell through the cracks

that was contained in r760866.

The following patch would fix this missing initialization:

Index: modules/ssl/ssl_engine_io.c
===================================================================
--- modules/ssl/ssl_engine_io.c (revision 771213)
+++ modules/ssl/ssl_engine_io.c (working copy)
@@ -1060,6 +1060,7 @@
     if (sslconn->is_proxy) {
         const char *hostname_note;

+        sc = mySrvConfig(sslconn->server);
         if ((n = SSL_connect(filter_ctx->pssl)) <= 0) {
             ap_log_cerror(APLOG_MARK, APLOG_INFO, 0, c,
                           "SSL Proxy connect failed");



Can I just correct this error and fix it in 2.2.x?

Yes, just fix it.

....Roy