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

Critical 9.8/10 vulnerability for this library #209

Closed
vwbusguy opened this issue Jun 6, 2022 · 8 comments
Closed

Critical 9.8/10 vulnerability for this library #209

vwbusguy opened this issue Jun 6, 2022 · 8 comments

Comments

@vwbusguy
Copy link

vwbusguy commented Jun 6, 2022

According to GitHub depandabot this library now has a 9.8/10 vulnerability at the current version (5.101.0) as well as a 7.5/10 high.

Here are the CVEs:
CVE-2019-15548
CVE-2019-15547

Here's the corresponding Rust Advisory.

@jeaye
Copy link
Owner

jeaye commented Jun 11, 2022

Thanks for sharing. I'd love a PR to fix each of these. For the latter one, the fn has been deprecated. For the former, there's been a comment there for a while, since the code was suspected to be incorrect. I haven't used Rust much in years, so I'm hoping someone else can pick this up. Perhaps @Ella-0?

@vwbusguy
Copy link
Author

I'd love to help, but the repo I linked to in my OP represents the current extent of my own Rust knowledge. I did that pancurses/ncurses-rs project a few years ago as a way to learn Rust myself, but I'm certainly more than a little rust-y on it since I haven't really touched it since then.

@Ella-0
Copy link
Collaborator

Ella-0 commented Jun 13, 2022

Perhaps @Ella-0?

Currently I'm in the middle of exams but I can have a look after they finish.

@jeaye
Copy link
Owner

jeaye commented Jun 16, 2022

Perhaps @Ella-0?

Currently I'm in the middle of exams but I can have a look after they finish.

Excellent. You're the best. Best of luck with your exams. :D

@jeaye
Copy link
Owner

jeaye commented Apr 10, 2024

Thank you for reporting. This is fixed in versions 6.0.0 and higher. 7bd2554

@jeaye jeaye closed this as completed Apr 10, 2024
@vwbusguy
Copy link
Author

Many thanks! I tested and added the fix to my repo and all seems happy now.

@correabuscar
Copy link
Contributor

correabuscar commented Apr 10, 2024

Many thanks! I tested and added the fix to my repo and all seems happy now.

I take it that (at least)compiles on Fedora? I'll add it to my virtualbox for testing...

EDIT: I'm trying to make it compile on Gentoo and NixOS, provided needed libs are already installed system-wide. Will PR after I make sure it works with a bunch of projects that use it (yours is on list too)
It does indeed compile on Fedora 39, if having ncurses-devel package.

@vwbusguy
Copy link
Author

vwbusguy commented Apr 10, 2024

It worked here with rust/Cargo 1.77 on Fedora 39. I didn't try in an RPM spec, but it should work, AFAICT. I'll get a bugzilla request to get the Fedora package updated, if it hasn't already been submitted.

EDIT: https://bugzilla.redhat.com/show_bug.cgi?id=2272332

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants