Make getParserError() to const (cppcheck warnings in rules_set code)

Author: airween

headers/modsecurity/rules_set.h

@@ -75,6 +75,7 @@ class RulesSet : public RulesSetProperties {
     int merge(RulesSet *rules);
 
     int evaluate(int phase, Transaction *transaction);
+    std::string getParserError() const;
     std::string getParserError();
 
     void debug(int level, const std::string &id, const std::string &uri,

src/rules_set.cc

@@ -101,10 +101,14 @@ int RulesSet::load(const char *plainRules) {
 }
 
 
-std::string RulesSet::getParserError() {
+std::string RulesSet::getParserError() const {
     return this->m_parserError.str();
 }
 
+std::string RulesSet::getParserError() {
+    return static_cast<const RulesSet&>(*this).getParserError();
+}
+
 void RulesSet::cleanMatchedVars(Transaction *trans) {
     ms_dbg_a(trans, 9, "Matched vars cleaned.");
     // cppcheck-suppress ctunullpointer
@@ -119,7 +123,7 @@ int RulesSet::evaluate(int phase, Transaction *t) {
        return 0;
     }
 
-    Rules *rules = m_rulesSetPhases[phase];
+    const Rules *rules = m_rulesSetPhases[phase];
 
     ms_dbg_a(t, 9, "This phase consists of " \
         + std::to_string(rules->size()) + " rule(s).");