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] ncurses vulnerability - CVE-2019-15548 #656

Closed
Terkwood opened this issue Jun 19, 2022 · 6 comments
Closed

[BUG] ncurses vulnerability - CVE-2019-15548 #656

Terkwood opened this issue Jun 19, 2022 · 6 comments
Labels

Comments

@Terkwood
Copy link

Describe the bug
ncurses is subject to the vulnerability listed in https://nvd.nist.gov/vuln/detail/CVE-2019-15548

Environment

  • Cursive commit 0c6669d (at least)
@Terkwood Terkwood added the bug label Jun 19, 2022
@gyscos
Copy link
Owner

gyscos commented Jun 19, 2022

Thanks for the report!

I don't think we use the affected functions, but it doesn't hurt to update to the latest version of fixed.

@dbrgn
Copy link
Contributor

dbrgn commented Sep 27, 2022

More information can be found here: https://rustsec.org/advisories/RUSTSEC-2019-0006

Affected functions:

  • ncurses::instr
  • ncurses::mvprintw
  • ncurses::mvwinstr
  • ncurses::mvwprintw
  • ncurses::printw

I don't think we use the affected functions

Yeah, I couldn't find any usage of an affected function either.

but it doesn't hurt to update to the latest version of fixed

So far there's no fix available: jeaye/ncurses-rs#209

@Terkwood
Copy link
Author

Will close this for now. Thank you

@correabuscar
Copy link
Contributor

correabuscar commented Apr 9, 2024

Would be nice to get cursive to use the version 6 of ncurses-rs possibly after jeaye/ncurses-rs#209 is closed(although it seems at least some issues were addressed),
but to do so at this time would require the following 2 patches: one would apply to ncurses-rs and one to cursive to facilitate this transition, but of course ncurses-rs would have to update first and release on crates.io, only after cursive would patch to use it:
jeaye/ncurses-rs#201 (comment)

@vwbusguy
Copy link

Getting this would also help us ship the ncurses library update in Fedora as well: https://bugzilla.redhat.com/show_bug.cgi?id=2272332

@correabuscar
Copy link
Contributor

I'll submit a PR later after the ncurses-rs one gets in first: jeaye/ncurses-rs#218
(I'll do one for pancurses too, already got them both working)

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

No branches or pull requests

5 participants