Skip to content

Add RUSTSEC advisory for git2#2620

Merged
djc merged 3 commits intorustsec:mainfrom
BoLu1225:git2-advisory
Feb 4, 2026
Merged

Add RUSTSEC advisory for git2#2620
djc merged 3 commits intorustsec:mainfrom
BoLu1225:git2-advisory

Conversation

@BoLu1225
Copy link
Contributor

@BoLu1225 BoLu1225 commented Feb 2, 2026

Have confirmed with the maintainer and fixed the bug in latest release

@djc
Copy link
Member

djc commented Feb 3, 2026

@ehuss do you agree that publishing an advisory for this is useful? Should it be informational = "unsound"?

Copy link
Contributor

@cuviper cuviper left a comment

Choose a reason for hiding this comment

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

FYI we are discussing in the security response team whether this warrants a full CVE, but it didn't come to our attention until after the fact. I personally think at least informational = "unsound" is reasonable though, and I suppose we can update rustsec if we decide to go further.

Co-authored-by: Josh Stone <cuviper@gmail.com>
@djc
Copy link
Member

djc commented Feb 4, 2026

FYI we are discussing in the security response team whether this warrants a full CVE, but it didn't come to our attention until after the fact. I personally think at least informational = "unsound" is reasonable though, and I suppose we can update rustsec if we decide to go further.

Thanks. @BoLu1225 can you add the informational metadata?

@djc djc merged commit eca995a into rustsec:main Feb 4, 2026
1 check passed
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.

3 participants