Skip to content

Support dpkginfo name pattern operations#2354

Open
officialasishkumar wants to merge 2 commits into
OpenSCAP:mainfrom
officialasishkumar:fix_2338
Open

Support dpkginfo name pattern operations#2354
officialasishkumar wants to merge 2 commits into
OpenSCAP:mainfrom
officialasishkumar:fix_2338

Fix dpkginfo review findings

a4cba45
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 1, 2026 in 26s

Annotations

Check failure on line 266 in src/OVAL/probes/unix/linux/dpkginfo-helper.c

See this annotation in the file changed.

@sonarqubecloud 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=OpenSCAP_openscap&issues=AZ8cpZc6vAm_7qDFniDY&open=AZ8cpZc6vAm_7qDFniDY&pullRequest=2354

Check warning on line 196 in src/OVAL/probes/unix/linux/dpkginfo-helper.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define each identifier in a dedicated statement.

See more on https://sonarcloud.io/project/issues?id=OpenSCAP_openscap&issues=AZ8cpZc6vAm_7qDFniDb&open=AZ8cpZc6vAm_7qDFniDb&pullRequest=2354

Check failure on line 271 in src/OVAL/probes/unix/linux/dpkginfo-helper.c

See this annotation in the file changed.

@sonarqubecloud 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=OpenSCAP_openscap&issues=AZ8cpZc6vAm_7qDFniDa&open=AZ8cpZc6vAm_7qDFniDa&pullRequest=2354

Check failure on line 262 in src/OVAL/probes/unix/linux/dpkginfo-helper.c

See this annotation in the file changed.

@sonarqubecloud 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=OpenSCAP_openscap&issues=AZ8cpZc6vAm_7qDFniDX&open=AZ8cpZc6vAm_7qDFniDX&pullRequest=2354

Check warning on line 271 in src/OVAL/probes/unix/linux/dpkginfo-helper.c

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=OpenSCAP_openscap&issues=AZ8cpZc6vAm_7qDFniDc&open=AZ8cpZc6vAm_7qDFniDc&pullRequest=2354

Check failure on line 193 in src/OVAL/probes/unix/linux/dpkginfo-helper.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=OpenSCAP_openscap&issues=AZ8cpZc6vAm_7qDFniDW&open=AZ8cpZc6vAm_7qDFniDW&pullRequest=2354

Check failure on line 268 in src/OVAL/probes/unix/linux/dpkginfo-helper.c

See this annotation in the file changed.

@sonarqubecloud 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=OpenSCAP_openscap&issues=AZ8cpZc6vAm_7qDFniDZ&open=AZ8cpZc6vAm_7qDFniDZ&pullRequest=2354

Check failure on line 47 in tests/probes/dpkginfo/test_probes_dpkginfo.sh

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use '[[' instead of '[' for conditional tests. The '[[' construct is safer and more feature-rich.

See more on https://sonarcloud.io/project/issues?id=OpenSCAP_openscap&issues=AZ8cpZhQvAm_7qDFniDd&open=AZ8cpZhQvAm_7qDFniDd&pullRequest=2354