diff --git a/apache2/re.c b/apache2/re.c index 51a4fea5..83f4d59a 100644 --- a/apache2/re.c +++ b/apache2/re.c @@ -1889,12 +1889,17 @@ static apr_status_t msre_ruleset_process_phase_(msre_ruleset *ruleset, modsec_re } } else if (rc < 0) { - const char *id = "", *msg = ""; - if (rule->actionset) { - if (rule->actionset->id) id = rule->actionset->id; - if (rule->actionset->msg) msg = rule->actionset->msg; - } - msr_log(msr, 1, "Rule processing failed (id=%s, msg=%s).", id, msg); + const char *id = ""; + const char *msg = ""; + if (rule->actionset) { + if (rule->actionset->id) { + id = rule->actionset->id; + } + if (rule->actionset->msg) { + msg = rule->actionset->msg; + } + } + msr_log(msr, 1, "Rule processing failed (id=%s, msg=%s).", id, msg); if (msr->txcfg->reqintercept_oe == 1) { apr_table_clear(msr->matched_vars); @@ -1924,12 +1929,17 @@ static apr_status_t msre_ruleset_process_phase_(msre_ruleset *ruleset, modsec_re } } else { - const char *id = "", *msg = ""; - if (rule->actionset) { - if (rule->actionset->id) id = rule->actionset->id; - if (rule->actionset->msg) msg = rule->actionset->msg; - } - msr_log(msr, 1, "Rule processing failed with unknown return code: %d (id=%s, msg=%s).", rc, id, msg); + const char *id = ""; + const char *msg = ""; + if (rule->actionset) { + if (rule->actionset->id) { + id = rule->actionset->id; + } + if (rule->actionset->msg) { + msg = rule->actionset->msg; + } + } + msr_log(msr, 1, "Rule processing failed with unknown return code: %d (id=%s, msg=%s).", rc, id, msg); apr_table_clear(msr->matched_vars); return -1; }