summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStefan Eissing <icing@apache.org>2022-02-07 15:40:46 +0100
committerStefan Eissing <icing@apache.org>2022-02-07 15:40:46 +0100
commit711bf3ee18283cc12d09aaaf07479696797cbbe8 (patch)
treee5328a489191d8785ccb8de59407ca5df94c6c09
parent *) core/mod_ssl/mpm_event: reverting changes to nonblocing SSL handshakes (diff)
downloadapache2-711bf3ee18283cc12d09aaaf07479696797cbbe8.tar.xz
apache2-711bf3ee18283cc12d09aaaf07479696797cbbe8.zip
*) mod_md: the status description in MDomain's JSON, exposed in the
md-status handler (if configure) did sometimes not carry the correct message when certificates needed renew. [Stefan Eissing] git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1897819 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r--changes-entries/mod_md_status_memory.txt4
-rw-r--r--modules/md/md_reg.c9
2 files changed, 11 insertions, 2 deletions
diff --git a/changes-entries/mod_md_status_memory.txt b/changes-entries/mod_md_status_memory.txt
new file mode 100644
index 0000000000..2ebd10ff89
--- /dev/null
+++ b/changes-entries/mod_md_status_memory.txt
@@ -0,0 +1,4 @@
+ *) mod_md: the status description in MDomain's JSON, exposed in the
+ md-status handler (if configure) did sometimes not carry the correct
+ message when certificates needed renew.
+ [Stefan Eissing]
diff --git a/modules/md/md_reg.c b/modules/md/md_reg.c
index 0fe6fff305..4fba9c7ae4 100644
--- a/modules/md/md_reg.c
+++ b/modules/md/md_reg.c
@@ -208,7 +208,12 @@ static apr_status_t state_init(md_reg_t *reg, apr_pool_t *p, md_t *md)
if (md->renew_window == NULL) md->renew_window = reg->renew_window;
if (md->warn_window == NULL) md->warn_window = reg->warn_window;
-
+
+ if (md->domains && md->domains->pool != p) {
+ md_log_perror(MD_LOG_MARK, MD_LOG_ERR, 0, p,
+ "md{%s}: state_init called with foreign pool", md->name);
+ }
+
for (i = 0; i < md_cert_count(md); ++i) {
spec = md_pkeys_spec_get(md->pks, i);
md_log_perror(MD_LOG_MARK, MD_LOG_TRACE2, rv, p,
@@ -902,7 +907,7 @@ apr_status_t md_reg_sync_finish(md_reg_t *reg, md_t *md, apr_pool_t *p, apr_pool
md->ca_proto = MD_PROTO_ACME;
}
- rv = state_init(reg, ptemp, md);
+ rv = state_init(reg, p, md);
if (APR_SUCCESS != rv) goto leave;
md_log_perror(MD_LOG_MARK, MD_LOG_DEBUG, rv, ptemp, "loading md %s", md->name);