jeremylong / DependencyCheck

OWASP dependency-check is a software composition analysis utility that detects publicly disclosed vulnerabilities in application dependencies.
https://owasp.org/www-project-dependency-check/
Apache License 2.0
6.32k stars 1.26k forks source link

[FP]: CVE-2016-10543, which applies to the call module of the hapijs/hapi.js framework, gets falgged on ktor dependencies with `call` in the name #6694

Closed volkert-fastned closed 3 months ago

volkert-fastned commented 3 months ago

Package URl

pkg:maven/io.ktor/ktor-server-call-logging-jvm@2.3.11

CPE

cpe:2.3:a:call_project:call:2.3.11:::::::*

CVE

CVE-2016-10543

ODC Integration

{"label"=>"Gradle Plugin"}

ODC Version

9.2.0

Description

Vulnerability CVE-2016-10543, which applies to the call module of hapi.js (a Node.js library) gets flagged on Ktor dependencies that have call in the name:

ktor-server-call-logging-jvm-2.3.11-sources.jar (pkg:maven/io.ktor/ktor-server-call-logging-jvm@2.3.11, cpe:2.3:a:call_project:call:2.3.11:*:*:*:*:*:*:*) : CVE-2016-10543

Strangely enough, this suddenly got flagged (along with a bunch of other vulnerabilities) while I tried to upgrade a project from Kotlin 1.9.x to Kotlin 2.0. But the dependencies that suddenly got flagged with that upgrade weren't related to the Kotlin 2.0 upgrade.

Also worth noting: according to NIST, the CPE should in fact be cpe:2.3:a:call_project:call:*:*:*:*:*:node.js:*:*, but the CPE that the DendencyCheck Gradle plugin flags has a wildcard in place of the node.js part. Perhaps that's what's causing this false positive?

github-actions[bot] commented 3 months ago

Maven Coordinates

<dependency>
   <groupId>io.ktor</groupId>
   <artifactId>ktor-server-call-logging-jvm</artifactId>
   <version>2.3.11</version>
</dependency>

Suppression rule:

<suppress base="true">
   <notes><![CDATA[
   FP per issue #6694
   ]]></notes>
   <packageUrl regex="true">^pkg:maven/io\.ktor/ktor-server-call-logging-jvm@.*$</packageUrl>
   <cpe>cpe:/a:call_project:call</cpe>
</suppress>

Link to test results: https://github.com/jeremylong/DependencyCheck/actions/runs/9268548184

volkert-fastned commented 3 months ago

This is one of multiple FPs of Node.js vulnerabilities that suddenly got flagged on Java/JVM dependencies while I was upgrading Kotlin from 1.9.x to 2.0, apparently because the CPE had an asterisk in the third position, which should have contained the string node.js, according to NIST:

jeremylong commented 3 months ago

approved

github-actions[bot] commented 3 months ago

Suppress rule has been added to the generatedSuppressions branch.