mirror of
https://github.com/owasp-modsecurity/ModSecurity.git
synced 2025-08-14 13:56:01 +03:00
Implement sonarcloud suggestions
This commit is contained in:
parent
9f5dc200ba
commit
1f419bba8f
@ -124,7 +124,7 @@ class ReadingLogsViaRuleMessage {
|
|||||||
m_rules(rules)
|
m_rules(rules)
|
||||||
{ }
|
{ }
|
||||||
|
|
||||||
int process() {
|
int process() const {
|
||||||
pthread_t threads[NUM_THREADS];
|
pthread_t threads[NUM_THREADS];
|
||||||
int i;
|
int i;
|
||||||
struct data_ms dms;
|
struct data_ms dms;
|
||||||
|
@ -133,7 +133,7 @@ bool RuleWithOperator::executeOperatorAt(Transaction *trans, const std::string &
|
|||||||
|
|
||||||
void RuleWithOperator::getVariablesExceptions(Transaction *t,
|
void RuleWithOperator::getVariablesExceptions(Transaction *t,
|
||||||
variables::Variables *exclusion, variables::Variables *addition) {
|
variables::Variables *exclusion, variables::Variables *addition) {
|
||||||
for (auto &a : t->m_rules->m_exceptions.m_variable_update_target_by_tag) { // cppcheck-suppress ctunullpointer
|
for (const auto &a : t->m_rules->m_exceptions.m_variable_update_target_by_tag) { // cppcheck-suppress ctunullpointer
|
||||||
if (containsTag(*a.first.get(), t) == false) {
|
if (containsTag(*a.first.get(), t) == false) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
@ -147,7 +147,7 @@ void RuleWithOperator::getVariablesExceptions(Transaction *t,
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
for (auto &a : t->m_rules->m_exceptions.m_variable_update_target_by_msg) {
|
for (const auto &a : t->m_rules->m_exceptions.m_variable_update_target_by_msg) {
|
||||||
if (containsMsg(*a.first.get(), t) == false) {
|
if (containsMsg(*a.first.get(), t) == false) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
@ -161,7 +161,7 @@ void RuleWithOperator::getVariablesExceptions(Transaction *t,
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
for (auto &a : t->m_rules->m_exceptions.m_variable_update_target_by_id) {
|
for (const auto &a : t->m_rules->m_exceptions.m_variable_update_target_by_id) {
|
||||||
if (m_ruleId != a.first) {
|
if (m_ruleId != a.first) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user