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

chore(audit): ignore RUSTSEC-2025-0023 #2565

Merged
merged 4 commits into from
Apr 7, 2025
Merged

Conversation

mxinden
Copy link
Collaborator

@mxinden mxinden commented Apr 7, 2025

RUSTSEC-2025-0023 discloses a vulnerability in Tokio: "Broadcast channel calls clone in parallel, but does not require Sync".

https://rustsec.org/advisories/RUSTSEC-2025-0023

Tokio is only used in neqo-bin. neqo-bin does not make use of Tokio's broadcast channels. neqo-bin is not used by Firefox.

All Tokio versions with a fix for RUSTSEC-2025-0023 require a libc version bump, i.e. require us to use a libc version other than what mozilla-central uses.

Thus, instead of updating Tokio, this commit simply ignores the CVE.

RUSTSEC-2025-0023 discloses a vulnerability in Tokio: "Broadcast channel
calls clone in parallel, but does not require `Sync`".

https://rustsec.org/advisories/RUSTSEC-2025-0023

Tokio is only used in neqo-bin. neqo-bin does not make use of Tokio's broadcast
channels. neqo-bin is not used by Firefox.

All Tokio versions with a fix for RUSTSEC-2025-0023 require a libc
version bump, i.e. require us to use a libc version other than what
mozilla-central uses.

Thus, instead of updating Tokio, this commit simply ignores the CVE.
@mxinden mxinden marked this pull request as ready for review April 7, 2025 10:32
@mxinden
Copy link
Collaborator Author

mxinden commented Apr 7, 2025

CI failure due to #2422.

Copy link

github-actions bot commented Apr 7, 2025

Failed Interop Tests

QUIC Interop Runner, client vs. server

neqo-latest as client

neqo-latest as server

All results

Succeeded Interop Tests

QUIC Interop Runner, client vs. server

neqo-latest as client

neqo-latest as server

Unsupported Interop Tests

QUIC Interop Runner, client vs. server

neqo-latest as client

neqo-latest as server

@mxinden mxinden enabled auto-merge April 7, 2025 12:30
@mxinden mxinden added this pull request to the merge queue Apr 7, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 7, 2025
@mxinden mxinden added this pull request to the merge queue Apr 7, 2025
Merged via the queue into mozilla:main with commit 782cf68 Apr 7, 2025
78 of 81 checks passed
@mxinden mxinden deleted the ignore-cve branch April 7, 2025 13:18
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.

2 participants