Suppress cppcheck false positive, as copy of string is required to av…

…oid use after free
eduar-hte · May 4, 2024 · 04e424c · 04e424c
1 parent 3862695
commit 04e424c
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/modsecurity.cc b/src/modsecurity.cc
@@ -262,8 +262,10 @@ int ModSecurity::processContentOffset(const char *content, size_t len,
         std::string value;
         yajl_gen_map_open(g);
         vars.pop_back();
+        // cppcheck-suppress redundantCopyLocalConst ; copy required to avoid use after free, see commit 7cb67b0
         const std::string startingAt = vars.back().str();
         vars.pop_back();
+        // cppcheck-suppress redundantCopyLocalConst ; copy required to avoid use after free, see commit 7cb67b0
         const std::string size = vars.back().str();
         vars.pop_back();
         yajl_gen_string(g,
@@ -346,8 +348,10 @@ int ModSecurity::processContentOffset(const char *content, size_t len,
             strlen("highlight"));
         yajl_gen_map_open(g);
         ops.pop_back();
+        // cppcheck-suppress redundantCopyLocalConst ; copy required to avoid use after free, see commit 7cb67b0
         const std::string startingAt = ops.back().str();
         ops.pop_back();
+        // cppcheck-suppress redundantCopyLocalConst ; copy required to avoid use after free, see commit 7cb67b0
         const std::string size = ops.back().str();
         ops.pop_back();
         yajl_gen_string(g,