From d0a63aac03bb1207f950a164ecf01806c1a38637 Mon Sep 17 00:00:00 2001 From: Robert Paprocki Date: Tue, 10 Apr 2018 09:01:34 -0700 Subject: [PATCH] Define m_secmarker_skipped as an integer type There's no reason to treat this this as a double, since it represents a human-readable data value that is only meaningful as an integer. In doing so we write cleaner audit logs and save a small amount of space. --- headers/modsecurity/rules.h | 2 +- test/test-cases/regression/secmarker.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/headers/modsecurity/rules.h b/headers/modsecurity/rules.h index ea2596cd..c574b136 100644 --- a/headers/modsecurity/rules.h +++ b/headers/modsecurity/rules.h @@ -80,7 +80,7 @@ class Rules : public RulesProperties { private: int m_referenceCount; - double m_secmarker_skipped; + uint8_t m_secmarker_skipped; }; #endif diff --git a/test/test-cases/regression/secmarker.json b/test/test-cases/regression/secmarker.json index 337f5182..bf5354c9 100644 --- a/test/test-cases/regression/secmarker.json +++ b/test/test-cases/regression/secmarker.json @@ -110,7 +110,7 @@ }, "expected": { "audit_log": "", - "debug_log": "Out of a SecMarker after skip 6.000000 rules.", + "debug_log": "Out of a SecMarker after skip 6 rules.", "error_log": "" }, "rules": [