Minor code cleanup.

This commit is contained in:
ivanr
2008-06-05 14:00:28 +00:00
parent 81d98de283
commit b2119411dd

View File

@@ -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) { if (actionset->auditlog != 0) {
msr->is_relevant++; 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); int log_level = (actionset->log == 0 ? 4 : 2);
msc_alert(msr, log_level, actionset, "Warning.", message); 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 * which will mess up noauditlog. We need to compensate for this
* so that we do not increment twice when auditlog is enabled and * so that we do not increment twice when auditlog is enabled and
* prevent incrementing when auditlog is disabled. * 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) { if (log_level <= 3) {
msr->is_relevant--; msr->is_relevant--;
} }
return; return;
} }