Commit 81a86d2

committed

Suppress cppcheck false positive, as copy of string is required to avoid use after free

1 parent 361d10a commit 81a86d2Copy full SHA for 81a86d2

File tree

-0

lines changed

-0

lines changed

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -262,8 +262,10 @@ int ModSecurity::processContentOffset(const char *content, size_t len,`
`262`	`262`	`std::string value;`
`263`	`263`	`yajl_gen_map_open(g);`
`264`	`264`	`vars.pop_back();`
	`265`	`+ // cppcheck-suppress redundantCopyLocalConst ; copy required to avoid use after free, see commit 7cb67b0`
`265`	`266`	`const std::string startingAt = vars.back().str();`
`266`	`267`	`vars.pop_back();`
	`268`	`+ // cppcheck-suppress redundantCopyLocalConst ; copy required to avoid use after free, see commit 7cb67b0`
`267`	`269`	`const std::string size = vars.back().str();`
`268`	`270`	`vars.pop_back();`
`269`	`271`	`yajl_gen_string(g,`
`@@ -346,8 +348,10 @@ int ModSecurity::processContentOffset(const char *content, size_t len,`
`346`	`348`	`strlen("highlight"));`
`347`	`349`	`yajl_gen_map_open(g);`
`348`	`350`	`ops.pop_back();`
	`351`	`+ // cppcheck-suppress redundantCopyLocalConst ; copy required to avoid use after free, see commit 7cb67b0`
`349`	`352`	`const std::string startingAt = ops.back().str();`
`350`	`353`	`ops.pop_back();`
	`354`	`+ // cppcheck-suppress redundantCopyLocalConst ; copy required to avoid use after free, see commit 7cb67b0`
`351`	`355`	`const std::string size = ops.back().str();`
`352`	`356`	`ops.pop_back();`
`353`	`357`	`yajl_gen_string(g,`

Comments

(0)