github_mirrors/ModSecurity
3
0
Fork 1
mirror of https://github.com/owasp-modsecurity/ModSecurity.git synced 2025-11-16 17:41:52 +03:00
Code Issues Packages Projects Releases Wiki Activity

Make all "rule id" variables of type RuleId

Browse Source 
Previously, ModSecurity inconsistently used RuleId, int and double for
rule id variables in different places.
This commit is contained in:
WGH
2020-07-24 20:12:35 +03:00
committed by Felipe Zimmerle
parent d393b4734b
commit b32a169d2a
6 changed files with 62 additions and 61 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/actions/rule_id.cc
View File

@@ -26,18 +26,16 @@ namespace actions {
bool RuleId::init(std::string *error) {
std::string a = m_parserPayload;
try {
m_ruleId = std::stod(a);
} catch (...) {
std::istringstream iss(a);
iss >> m_ruleId;
if (iss.fail()) {
m_ruleId = 0;
error->assign("The input \"" + a + "\" does not " \
"seems to be a valid rule id.");
return false;
}
std::ostringstream oss;
oss << std::setprecision(40) << m_ruleId;
if (a != oss.str() || m_ruleId < 0) {
if (a != std::to_string(m_ruleId) || m_ruleId < 0) {
error->assign("The input \"" + a + "\" does not seems " \
"to be a valid rule id.");
return false;