Added missing override keyword as reported by cppcheck 2.14

Author: eduar-hte

headers/modsecurity/rule_with_actions.h

@@ -43,7 +43,7 @@ class RuleWithActions : public Rule {
         std::unique_ptr<std::string> fileName,
         int lineNumber);
 
-    ~RuleWithActions();
+    ~RuleWithActions() override;
 
     RuleWithActions(const RuleWithActions& r)
         : Rule(r),

headers/modsecurity/rule_with_operator.h

@@ -45,7 +45,7 @@ class RuleWithOperator : public RuleWithActions {
         std::unique_ptr<std::string> fileName,
         int lineNumber);
 
-    virtual ~RuleWithOperator();
+    ~RuleWithOperator() override;
 
     bool evaluate(Transaction *transaction,
         std::shared_ptr<RuleMessage> rm) override;

src/actions/exec.h

@@ -34,7 +34,7 @@ class Exec : public Action {
         : Action(action),
         m_script("") { }
 
-    ~Exec() { }
+    ~Exec() override { }
 
     bool evaluate(RuleWithActions *rule, Transaction *transaction) override;
     bool init(std::string *error) override;

src/collection/backend/in_memory-per_process.h

@@ -75,7 +75,7 @@ class InMemoryPerProcess :
     public Collection {
  public:
     explicit InMemoryPerProcess(const std::string &name);
-    ~InMemoryPerProcess();
+    ~InMemoryPerProcess() override;
     void store(std::string key, std::string value);
 
     bool storeOrUpdateFirst(const std::string &key,

src/operators/fuzzy_hash.h

@@ -42,7 +42,7 @@ class FuzzyHash : public Operator {
         : Operator("FuzzyHash", std::move(param)),
         m_threshold(0),
         m_head(NULL) { }
-    ~FuzzyHash();
+    ~FuzzyHash() override;
 
     bool evaluate(Transaction *transaction, const std::string &std) override;
 

src/operators/pm.h

@@ -40,7 +40,7 @@ class Pm : public Operator {
         : Operator(n, std::move(param)) {
         m_p = acmp_create(0);
     }
-    ~Pm();
+    ~Pm() override;
     bool evaluate(Transaction *transaction, RuleWithActions *rule,
         const std::string &str,
         std::shared_ptr<RuleMessage> ruleMessage) override;

src/operators/rx.h

@@ -42,7 +42,7 @@ class Rx : public Operator {
             m_couldContainsMacro = true;
         }
 
-    ~Rx() {
+    ~Rx() override {
         if (m_string->m_containsMacro == false && m_re != NULL) {
             delete m_re;
             m_re = NULL;

src/operators/rx_global.h

@@ -42,7 +42,7 @@ class RxGlobal : public Operator {
             m_couldContainsMacro = true;
         }
 
-    ~RxGlobal() {
+    ~RxGlobal() override {
         if (m_string->m_containsMacro == false && m_re != NULL) {
             delete m_re;
             m_re = NULL;

src/operators/validate_dtd.h

@@ -57,7 +57,7 @@ class ValidateDTD : public Operator {
     explicit ValidateDTD(std::unique_ptr<RunTimeString> param)
         : Operator("ValidateDTD", std::move(param)) { }
 #ifdef WITH_LIBXML2
-    ~ValidateDTD() { }
+    ~ValidateDTD() override { }
 
     bool evaluate(Transaction *transaction, const std::string  &str) override;
     bool init(const std::string &file, std::string *error) override;

src/operators/validate_schema.h

@@ -38,7 +38,7 @@ class ValidateSchema : public Operator {
     /** @ingroup ModSecurity_Operator */
     explicit ValidateSchema(std::unique_ptr<RunTimeString> param)
         : Operator("ValidateSchema", std::move(param)) { }
-    ~ValidateSchema() { }
+    ~ValidateSchema() override { }
 #ifdef WITH_LIBXML2
 
     bool evaluate(Transaction *transaction, const std::string  &str) override;

src/operators/verify_cc.h

@@ -45,7 +45,7 @@ class VerifyCC : public Operator {
         m_pc(NULL),
         m_pce(NULL) { }
 #endif
-    ~VerifyCC();
+    ~VerifyCC() override;
 
     bool evaluate(Transaction *t, RuleWithActions *rule,
         const std::string& input,

src/operators/verify_cpf.h

@@ -39,7 +39,7 @@ class VerifyCPF : public Operator {
         m_re = new Regex(m_param);
     }
 
-    ~VerifyCPF() {
+    ~VerifyCPF() override {
         delete m_re;
     }
 

src/operators/verify_ssn.h

@@ -39,7 +39,7 @@ class VerifySSN : public Operator {
         m_re = new Regex(m_param);
     }
 
-    ~VerifySSN() {
+    ~VerifySSN() override {
         delete m_re;
     }
 

src/operators/verify_svnr.h

@@ -25,7 +25,7 @@ class VerifySVNR : public Operator {
         m_re = new Regex(m_param);
     }
 
-    ~VerifySVNR() {
+    ~VerifySVNR() override {
         delete m_re;
     }
 

test/regression/custom_debug_log.h

@@ -26,7 +26,7 @@ namespace modsecurity_test {
 class CustomDebugLog : public modsecurity::debug_log::DebugLog {
  public:
     CustomDebugLog *new_instance();
-    ~CustomDebugLog();
+    ~CustomDebugLog() override;
 
     void write(int level, const std::string& message) override;
     void write(int level, const std::string &id,