Fix memory leak #3510

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open

JonathanBerrew wants to merge 4 commits into owasp-modsecurity:v2/master from JonathanBerrew:fix-memory-leak

+81 −60

Open

Fix memory leak #3510

Add missing const

SonarQubeCloud / SonarCloud Code Analysis failed Apr 9, 2026 in 27s

Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check failure on line 272 in apache2/apache2_config.c

sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest more than 3 if|for|do|while|switch statements.

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

Check failure on line 197 in apache2/apache2_config.c

sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 85 to the 25 allowed.

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

Check failure on line 817 in apache2/apache2_config.c

sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 57 to the 25 allowed.

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

View more details on SonarQubeCloud