Skip to content

Fix lexer rejection of '@' in ctl:ruleRemoveTarget actions (#3565)#3566

Open
Jitterx69 wants to merge 6 commits into
owasp-modsecurity:v3/masterfrom
Jitterx69:fix/issue-3565-ctl-removerule-xpath
Open

Fix lexer rejection of '@' in ctl:ruleRemoveTarget actions (#3565)#3566
Jitterx69 wants to merge 6 commits into
owasp-modsecurity:v3/masterfrom
Jitterx69:fix/issue-3565-ctl-removerule-xpath

Merge branch 'v3/master' into fix/issue-3565-ctl-removerule-xpath

14cf56f
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jun 16, 2026 in 2m 19s

Annotations

Check warning on line 8680 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract the assignment to "yy_n_chars" from this expression.

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

Check failure on line 199 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Global variables should be const.

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

Check warning on line 8392 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract the assignment to "filename" from this expression.

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

Check warning on line 8758 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant cast.

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

Check warning on line 9336 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract the assignment from this expression.

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

Check warning on line 9002 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract the assignment to "yy_c_buf_p" from this expression.

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

Check warning on line 1867 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "std::array" or "std::vector" instead of a C-style array.

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

Check warning on line 3362 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "std::array" or "std::vector" instead of a C-style array.

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

Check warning on line 8457 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract the assignment to "filename" from this expression.

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

Check warning on line 9281 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract the assignment to "yy_ch_buf" from this expression.

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

Check warning on line 8756 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these redundant parentheses.

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

Check failure on line 333 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Global variables should be const.

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

Check warning on line 8424 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract the assignment to "filename" from this expression.

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

Check warning on line 9328 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant cast.

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

Check warning on line 920 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "std::string" instead of a C-style char array.

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

Check failure on line 334 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Global variables should be const.

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

Check warning on line 8756 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these redundant parentheses.

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