Backport:

mod_ssl: The error message when SSLCertificateFile is missing should
at least give the name or position of the problematic virtual host
definition.
Submitted by: Stefan Fritsch <sf sfritsch.de>

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@815943 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/CHANGES b/CHANGES
index 5e65c6b2ce3d699527edb9c852c9cce823f25313..38ac518e4fa2fe034ffe4c3302434afb0fec18af 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -9,6 +9,10 @@ Changes with Apache 2.2.14
      mod_proxy_ftp: NULL pointer dereference on error paths.
      [Stefan Fritsch <sf fritsch.de>, Joe Orton]
 
+  *) mod_ssl: The error message when SSLCertificateFile is missing should
+     at least give the name or position of the problematic virtual host
+     definition. [Stefan Fritsch sf sfritsch.de]
+
   *) htdbm: Fix possible buffer overflow if dbm database has very
      long values.  PR 30586 [Dan Poirier]
 

diff --git a/STATUS b/STATUS
index 1893a4dacc7c27f11da9e8cf4eaaa4ebf93ccdd2..6af4a1fac7c66a2bf98d7c52164dbcdb45bec9b7 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -94,13 +94,6 @@ PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
    +1: covener, poirier
    +1: minfrin (with r814779 for compiler warning fix)
 
- * mod_ssl: The error message when SSLCertificateFile is missing should
-   at least give the name or position of the problematic virtual host
-   definition.
-   Trunk Patch: http://svn.apache.org/viewvc?rev=813105&view=rev
-   2.2.x Patch: http://people.apache.org/~minfrin/ssl_engine_pphrase.diff
-   +1: minfrin, rpluem, pquerna
-
 PATCHES PROPOSED TO BACKPORT FROM TRUNK:
   [ New proposals should be added at the end of the list ]
 

diff --git a/modules/ssl/ssl_engine_pphrase.c b/modules/ssl/ssl_engine_pphrase.c
index 76763e315b924060dd1c9bd15a0c94ed833ef564..dc49eff6e41c3f2b438d998d45fa4352218403a2 100644 (file)
--- a/modules/ssl/ssl_engine_pphrase.c
+++ b/modules/ssl/ssl_engine_pphrase.c
@@ -188,7 +188,8 @@ void ssl_pphrase_Handle(server_rec *s, apr_pool_t *p)
         if (sc->server->pks->cert_files[0] == NULL) {
             ap_log_error(APLOG_MARK, APLOG_ERR, 0, pServ,
                          "Server should be SSL-aware but has no certificate "
-                         "configured [Hint: SSLCertificateFile]");
+                         "configured [Hint: SSLCertificateFile] (%s:%d)",
+                         pServ->defn_name, pServ->defn_line_number);
             ssl_die();
         }
         algoCert = SSL_ALGO_UNKNOWN;