Skip to content
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

[WIP] Install deps before the scan #572

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

flor-master
Copy link
Collaborator

Install Dependencies Before Security Scan

Install dependencies before the Semgrep scanning project

@flor-master flor-master changed the title Install deps before the scan [WIP] Install deps before the scan Sep 27, 2023
Comment on lines +158 to +159
org.jetbrains.intellij.deps:trove4j:1.0.20200330=detekt,kotlinCompilerClasspath,kotlinCompilerPluginClasspathDebug,kotlinCompilerPluginClasspathDebugAndroidTest,kotlinCompilerPluginClasspathDebugUnitTest,kotlinCompilerPluginClasspathRelease,kotlinCompilerPluginClasspathReleaseUnitTest,kotlinKlibCommonizerClasspath
org.jetbrains.kotlin:kotlin-android-extensions-runtime:1.8.22=debugAndroidTestCompileClasspath,debugAndroidTestRuntimeClasspath,debugCompileClasspath,debugRuntimeClasspath,debugUnitTestCompileClasspath,debugUnitTestRuntimeClasspath,releaseCompileClasspath,releaseRuntimeClasspath,releaseUnitTestCompileClasspath,releaseUnitTestRuntimeClasspath
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Legal Risk:
org.jetbrains.intellij.deps:trove4j 1.0.20200330 was released under the LGPL-2.1 license, a license that has been flagged by your organization for consideration

Recommendation:
While merging is not directly blocked, it's best to pause and consider what it means to use this license before continuing. If you are unsure, reach out to your security team or Semgrep admin to address this issue.

Comment on lines +144 to +145
net.java.dev.jna:jna:5.6.0=detekt,kotlinCompilerClasspath,kotlinCompilerPluginClasspathDebug,kotlinCompilerPluginClasspathDebugAndroidTest,kotlinCompilerPluginClasspathDebugUnitTest,kotlinCompilerPluginClasspathRelease,kotlinCompilerPluginClasspathReleaseUnitTest,kotlinKlibCommonizerClasspath
org.bouncycastle:bcprov-jdk18on:1.72=debugUnitTestCompileClasspath,debugUnitTestRuntimeClasspath,releaseUnitTestCompileClasspath,releaseUnitTestRuntimeClasspath,testImplementationDependenciesMetadata
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Legal Risk:
net.java.dev.jna:jna 5.6.0 was released under the LGPL-2.1 license, a license that has been flagged by your organization for consideration

Recommendation:
While merging is not directly blocked, it's best to pause and consider what it means to use this license before continuing. If you are unsure, reach out to your security team or Semgrep admin to address this issue.

Comment on lines +85 to +86
com.fasterxml.woodstox:woodstox-core:6.2.4=dokkaGfmPartialRuntime,dokkaGfmRuntime,dokkaHtmlPartialRuntime,dokkaHtmlRuntime,dokkaJavadocPartialRuntime,dokkaJavadocRuntime,dokkaJekyllPartialRuntime,dokkaJekyllRuntime
com.google.android.material:material:1.9.0=apiDependenciesMetadata,debugAndroidTestCompileClasspath,debugAndroidTestRuntimeClasspath,debugCompileClasspath,debugRuntimeClasspath,debugUnitTestCompileClasspath,debugUnitTestRuntimeClasspath,implementationDependenciesMetadata,releaseCompileClasspath,releaseRuntimeClasspath,releaseUnitTestCompileClasspath,releaseUnitTestRuntimeClasspath
Copy link

@semgrep-app semgrep-app bot Sep 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Risk: woodstox-core 5.x before 5.4.0 and woodstox-core 6.x before 6.4.0 are vulnerable to Denial of Service (DOS) attacks through the DTD parsing functionality, which is used when serializing XML with FasterXML/woodstox. This happens because xstream, a dependency of woodstox, is vulnerable. This can be partially worked around by setting a stack size limit, but it's not guaranteed to prevent the issue, as per x-stream/xstream#316 (comment). Upgrade 5.x versions to 5.4.0 or 6.x versions to 6.4.0.

Fix: Upgrade this library to at least version 6.4.0 at vgs-collect-android/vgscollect/gradle.lockfile:85.

Reference(s): GHSA-fv22-xp26-mm9w, CVE-2022-40153

Ignore this finding from ssc-3f20a370-e458-4c69-a647-50011ca01c97.

Comment on lines +144 to +145
net.java.dev.jna:jna:5.6.0=detekt,kotlinCompilerClasspath,kotlinCompilerPluginClasspathDebug,kotlinCompilerPluginClasspathDebugAndroidTest,kotlinCompilerPluginClasspathDebugUnitTest,kotlinCompilerPluginClasspathRelease,kotlinCompilerPluginClasspathReleaseUnitTest,kotlinKlibCommonizerClasspath
org.bouncycastle:bcprov-jdk18on:1.72=debugUnitTestCompileClasspath,debugUnitTestRuntimeClasspath,releaseUnitTestCompileClasspath,releaseUnitTestRuntimeClasspath,testImplementationDependenciesMetadata
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Legal Risk:
net.java.dev.jna:jna 5.6.0 was released under the LGPL-2.1 license, a license that has been flagged by your organization for consideration

Recommendation:
While merging is not directly blocked, it's best to pause and consider what it means to use this license before continuing. If you are unsure, reach out to your security team or Semgrep admin to address this issue.

Comment on lines +158 to +159
org.jetbrains.intellij.deps:trove4j:1.0.20200330=detekt,kotlinCompilerClasspath,kotlinCompilerPluginClasspathDebug,kotlinCompilerPluginClasspathDebugAndroidTest,kotlinCompilerPluginClasspathDebugUnitTest,kotlinCompilerPluginClasspathRelease,kotlinCompilerPluginClasspathReleaseUnitTest,kotlinKlibCommonizerClasspath
org.jetbrains.kotlin:kotlin-android-extensions-runtime:1.8.22=debugAndroidTestCompileClasspath,debugAndroidTestRuntimeClasspath,debugCompileClasspath,debugRuntimeClasspath,debugUnitTestCompileClasspath,debugUnitTestRuntimeClasspath,releaseCompileClasspath,releaseRuntimeClasspath,releaseUnitTestCompileClasspath,releaseUnitTestRuntimeClasspath
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Legal Risk:
org.jetbrains.intellij.deps:trove4j 1.0.20200330 was released under the LGPL-2.1 license, a license that has been flagged by your organization for consideration

Recommendation:
While merging is not directly blocked, it's best to pause and consider what it means to use this license before continuing. If you are unsure, reach out to your security team or Semgrep admin to address this issue.

@semgrep-app
Copy link

semgrep-app bot commented Sep 28, 2023

Semgrep found 1 ssc-3f20a370-e458-4c69-a647-50011ca01c97 finding:

Risk: woodstox-core 5.x before 5.4.0 and woodstox-core 6.x before 6.4.0 are vulnerable to Denial of Service (DOS) attacks through the DTD parsing functionality, which is used when serializing XML with FasterXML/woodstox. This happens because xstream, a dependency of woodstox, is vulnerable. This can be partially worked around by setting a stack size limit, but it's not guaranteed to prevent the issue, as per x-stream/xstream#316 (comment). Upgrade 5.x versions to 5.4.0 or 6.x versions to 6.4.0.

Fix: Upgrade this library to at least version 6.4.0 at vgs-collect-android/gradle.lockfile:85.

Reference(s): GHSA-fv22-xp26-mm9w, CVE-2022-40153

Ignore this finding from ssc-3f20a370-e458-4c69-a647-50011ca01c97.

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.

None yet

2 participants