Skip to content

make rootpath and incpath consts for apr_filepath_root #3270

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.

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

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

Marcool04
Copy link

what

In standalone/config.c:

  • Make rootpath and incpath into const char (rather than just char)
  • replace rootpath where it was being written to, by another char, configfilepath

why

Avoids apr_filepath_root function to which these are passed throwing an "incompatible pointer type" warning/error (up to gcc13 it is a warning, from gcc14 an error).

references

Fixes #3173

Copy link

sonarqubecloud bot commented Oct 4, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

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

@marcstern marcstern merged commit d7f2be6 into owasp-modsecurity:v2/master Oct 4, 2024
40 of 41 checks passed
@Marcool04 Marcool04 deleted the v2/master branch October 4, 2024 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants