From b2119411ddeba287136e50b70d711bed46d7f4ad Mon Sep 17 00:00:00 2001 From: ivanr Date: Thu, 5 Jun 2008 14:00:28 +0000 Subject: [PATCH] Minor code cleanup. --- apache2/re.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/apache2/re.c b/apache2/re.c index 243f1451..f8ef7189 100644 --- a/apache2/re.c +++ b/apache2/re.c @@ -1616,7 +1616,7 @@ static void msre_perform_disruptive_actions(modsec_rec *msr, msre_rule *rule, } } - /* If "noauditlog" used do not mark the transaction relevant. */ + /* If "noauditlog" was used do not mark the transaction relevant. */ if (actionset->auditlog != 0) { msr->is_relevant++; } @@ -1633,7 +1633,7 @@ static void msre_perform_disruptive_actions(modsec_rec *msr, msre_rule *rule, int log_level = (actionset->log == 0 ? 4 : 2); msc_alert(msr, log_level, actionset, "Warning.", message); - /* However, this will mark the txn relevant again if it is <=3, + /* However, this will mark the txn relevant again if it is <= 3, * which will mess up noauditlog. We need to compensate for this * so that we do not increment twice when auditlog is enabled and * prevent incrementing when auditlog is disabled. @@ -1641,6 +1641,7 @@ static void msre_perform_disruptive_actions(modsec_rec *msr, msre_rule *rule, if (log_level <= 3) { msr->is_relevant--; } + return; }