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

[Bug, Vulnerability] CVE-2023-48795 #11936

Closed
2 of 3 tasks
VladislavDubrovenski opened this issue Feb 26, 2024 · 12 comments
Closed
2 of 3 tasks

[Bug, Vulnerability] CVE-2023-48795 #11936

VladislavDubrovenski opened this issue Feb 26, 2024 · 12 comments
Assignees
Labels
bug Something isn't working and you are sure it's a bug! dependencies Pull requests that update a dependency file docker Docker build scripts or images related
Milestone

Comments

@VladislavDubrovenski
Copy link

VladislavDubrovenski commented Feb 26, 2024

Search before asking

  • I had searched in the issues and found no similar issues.

Apache SkyWalking Component

OAP server (apache/skywalking)

What happened

Good day,

The scanner flags CVE-2023-48795 in OAP and UI that I am required to fix to continue using this great project..

The skywalking is deployed using a helm chart.

More information:
OAP:

  • fixedVersion: 0.17.0
    installedVersion: v0.0.0-20220411220226-7b82a4e95df4
    lastModifiedDate: "2024-01-29T09:15:42Z"
    links: []
    primaryLink: https://avd.aquasec.com/nvd/cve-2023-48795
    publishedDate: "2023-12-18T16:15:10Z"
    resource: golang.org/x/crypto
    score: 5.9
    severity: MEDIUM
    target: ""
    title: 'ssh: Prefix truncation attack on Binary Packet Protocol (BPP)'
    vulnerabilityID: CVE-2023-48795

UI:

  • fixedVersion: 0.9.6-2ubuntu0.22.04.2
    installedVersion: 0.9.6-2ubuntu0.22.04.1
    lastModifiedDate: "2024-01-29T09:15:42Z"
    links: []
    primaryLink: https://avd.aquasec.com/nvd/cve-2023-48795
    publishedDate: "2023-12-18T16:15:10Z"
    resource: libssh-4
    score: 5.9
    severity: MEDIUM
    target: ""
    title: 'ssh: Prefix truncation attack on Binary Packet Protocol (BPP)'
    vulnerabilityID: CVE-2023-4879

What you expected to happen

No vulnerability found

How to reproduce

Install skywalking via helm chart

Anything else

No response

Are you willing to submit a pull request to fix on your own?

  • Yes I am willing to submit a pull request on my own!

Code of Conduct

@VladislavDubrovenski VladislavDubrovenski added the bug Something isn't working and you are sure it's a bug! label Feb 26, 2024
@wu-sheng
Copy link
Member

wu-sheng commented Feb 26, 2024

We are not using Go and Python in OAP. I am not sure what you mean. This is a Java project.

@VladislavDubrovenski
Copy link
Author

@wu-sheng I understand that, I opened it for OAP. Also, it occurs to me that the image you use for deployment on k8s for UI has the vulnerability. Where should I report this? This vulnerability was present for almost 2 months and my only other remediation is to disable the skywalking which I really don't want to do as I have written custom scripts for it as well..

@wu-sheng
Copy link
Member

OK, if you mean images, you need to check whether it is from swctl or something? Because OAP and UI themselves are only Java based.

You could check the docker file, https://github.com/apache/skywalking/blob/master/docker/oap/Dockerfile

About the CLI, it is from https://github.com/apache/skywalking-cli.

@wu-sheng wu-sheng added docker Docker build scripts or images related dependencies Pull requests that update a dependency file labels Feb 26, 2024
@wu-sheng wu-sheng reopened this Feb 26, 2024
@wu-sheng
Copy link
Member

Or is this a Linux level CVE? You could repackage the whole thing and get the latest eclipse-temurin:11-jre.

@VladislavDubrovenski
Copy link
Author

@wu-sheng I appreciate you reopening this issue. Constantly repackaging every open-source solution would make it a nightmare from maintainability perspective. It seems that many open-source solutions suffer from this vulnerability(even though the library in question was not in use in many, but scans still flag it).

I have several examples where it was already fixed:

Jaeger: jaegertracing/jaeger#5016
Grafana: grafana/grafana#80316
ArgoCD: argoproj/argo-cd#17020
Prometheus: prometheus/prometheus#13512
And others

I understand that this causes certain inconveniences, and I apologize for that, but I had been required to address this particular vulnerability in the past month.

@wu-sheng
Copy link
Member

@kezhenxu94 Could you take a look? I think this may be either from base image or a kind of CLI side issues.

@wu-sheng
Copy link
Member

CLI fixed at apache/skywalking-cli#199

@wu-sheng wu-sheng added this to the 10.0.0 milestone Feb 27, 2024
@kezhenxu94
Copy link
Member

If the UI also has the CVE, then swctl might not be the only source of the CVE, as UI doesn't have swctl in it

@wu-sheng
Copy link
Member

Yes, I noticed that. I am not sure why UI could have this, maybe it is from default base Linux image? CLI has the version impact but maybe not the only source.

@heyanlong
Copy link
Member

I see eclipse-temurin:11-jre is built based on ubuntu:22.04, using OpenSSH 8.9 as default ssh in ubuntu:22.04

@wu-sheng
Copy link
Member

wu-sheng commented Mar 4, 2024

AFAIK, 11-jre image should be safe enough. https://hub.docker.com/layers/library/eclipse-temurin/11-jre/images/sha256-2a0e7b00897263d43b65f33962ca1299e6ce91a16c3dd09fbceff114d4c8c34a?context=explore

No high and critical issues. Our own side CLI has been removed from the next release.

We should be good now.

@wu-sheng wu-sheng closed this as completed Mar 4, 2024
@VladislavDubrovenski
Copy link
Author

Thank you for the quick remediation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working and you are sure it's a bug! dependencies Pull requests that update a dependency file docker Docker build scripts or images related
Projects
None yet
Development

No branches or pull requests

4 participants