Skip to content

clean cppcheck warnings in the iis/ ext/ and mlogc/ directories - #3596

Open
umprayz wants to merge 7 commits into
owasp-modsecurity:v2/masterfrom
umprayz:v2/cppcheck_0709
Open

clean cppcheck warnings in the iis/ ext/ and mlogc/ directories#3596
umprayz wants to merge 7 commits into
owasp-modsecurity:v2/masterfrom
umprayz:v2/cppcheck_0709

fixing iis build failure

e3c86f1
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jul 14, 2026 in 1m 1s

Quality Gate failed

Annotations

Check warning on line 479 in iis/moduleconfig.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not use the constructor's initializer list for data member "m_Config". Use the in-class initializer instead.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AZ9cOLuY0_vsReWVPh7R&open=AZ9cOLuY0_vsReWVPh7R&pullRequest=3596

Check warning on line 479 in iis/moduleconfig.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Field "m_Config" is initialized before field "m_bIsEnabled" and field "m_pszPath".

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AZ9cOLuY0_vsReWVPh7T&open=AZ9cOLuY0_vsReWVPh7T&pullRequest=3596

Check warning on line 478 in iis/moduleconfig.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not use the constructor's initializer list for data member "m_pszPath". Use the in-class initializer instead.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AZ9cOLuY0_vsReWVPh7S&open=AZ9cOLuY0_vsReWVPh7S&pullRequest=3596

Check warning on line 572 in iis/moduleconfig.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge this "if" statement with the enclosing one.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AZ9cOLuY0_vsReWVPh7U&open=AZ9cOLuY0_vsReWVPh7U&pullRequest=3596