From ad98ea71e237b453427d31845d276ebc133dcc48 Mon Sep 17 00:00:00 2001 From: Hans Zandbelt Date: Fri, 6 Sep 2024 14:46:35 +0200 Subject: [PATCH] fix child initialization routines using multiple vhosts; closes @1208 this fixes possible segmentation faults when using Redis and Metrics settings in vhosts; thanks @studersi Signed-off-by: Hans Zandbelt --- ChangeLog | 2 ++ src/mod_auth_openidc.c | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 8ab1afaa..3f2aeecc 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,7 @@ 09/06/2024 - allow overriding globally set OIDCCacheType back to shm in vhosts +- correct typo in child initialization routines when using multiple vhosts; closes @1208; thanks @studersi + this fixes possible segmentation faults when using Redis and Metrics settings in vhosts 09/05/2024 - fix OIDCCacheShmMax min/max settings; see #1260; thanks @bbartke diff --git a/src/mod_auth_openidc.c b/src/mod_auth_openidc.c index 6f4ac95f..7ffacf7f 100644 --- a/src/mod_auth_openidc.c +++ b/src/mod_auth_openidc.c @@ -2118,7 +2118,7 @@ static void oidc_child_init(apr_pool_t *p, server_rec *s) { server_rec *sp = s; while (sp != NULL) { oidc_cfg_t *cfg = (oidc_cfg_t *)ap_get_module_config(sp->module_config, &auth_openidc_module); - oidc_cfg_child_init(p, cfg, s); + oidc_cfg_child_init(p, cfg, sp); sp = sp->next; } apr_pool_cleanup_register(p, s, oidc_cleanup_child, apr_pool_cleanup_null);