Skip to content

Suppress Calcite CVE - #13119

Merged
abhishekagarwal87 merged 3 commits into
apache:masterfrom
adarshsanjeev:supress-calcite-cve
Sep 23, 2022
Merged

abhishekagarwal87 merged 3 commits into
apache:masterfrom
adarshsanjeev:supress-calcite-cve

Conversation

@adarshsanjeev

Copy link
Copy Markdown
Contributor

The vulnerability is in a builtin function in Calcite, EXTRACT_VALUE, which is not exposed in the Druid console, so druid should be unaffected.

  • Suppress calcite CVE.

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

<suppress>
<!-- avatica-server-1.17.0.jar -->
<notes><![CDATA[
file name: avatica-server-1.17.0.jar

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The vulnerability is in calcite-core. Does it need to be suppressed for avatica-server too?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I believe calcite is used by avatica server too, wouldn't it need to be suppressed separately?

@kfaraz kfaraz Sep 21, 2022 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think so. Could you try running the security check without the extra suppression?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the failure is being reported for avatica-server as well.

[ERROR] ----------------------------------------------------
[ERROR] .NET Assembly Analyzer could not be initialized and at least one 'exe' or 'dll' was scanned. The 'dotnet' executable could not be found on the path; either disable the Assembly Analyzer or add the path to dotnet core in the configuration.
[ERROR] ----------------------------------------------------
[ERROR] Failed to execute goal org.owasp:dependency-check-maven:7.0.4:aggregate (default-cli) on project druid: 
[ERROR] 
[ERROR] One or more dependencies were identified with vulnerabilities that have a CVSS score greater than or equal to '7.0': 
[ERROR] 
[ERROR] avatica-server-1.17.0.jar: CVE-2022-39135(9.8)
[ERROR] calcite-core-1.21.0.jar: CVE-2022-39135(9.8)
[ERROR] 
[ERROR] See the dependency-check report for more details.
[ERROR] -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

Comment thread owasp-dependency-check-suppressions.xml Outdated
@abhishekagarwal87
abhishekagarwal87 merged commit 306f612 into apache:master Sep 23, 2022
liam-verta pushed a commit to VertaAI/druid that referenced this pull request Sep 27, 2022
* Suppress Calcite CVE

* Update comment
liam-verta pushed a commit to VertaAI/druid that referenced this pull request Sep 28, 2022
* Suppress Calcite CVE

* Update comment
@kfaraz kfaraz added this to the 25.0 milestone Nov 22, 2022
riovic918data pushed a commit to riovic918data/druid that referenced this pull request Jun 12, 2026
* Suppress Calcite CVE

* Update comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants