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

Cargo audit warns about RUSTSEC-2019-0031 #24

Closed
gendx opened this issue Feb 6, 2020 · 2 comments
Closed

Cargo audit warns about RUSTSEC-2019-0031 #24

gendx opened this issue Feb 6, 2020 · 2 comments
Assignees

Comments

@gendx
Copy link
Collaborator

gendx commented Feb 6, 2020

Expected Behavior

Cargo audit doesn't find any advisory.

Actual Behavior

RUSTSEC-2019-0031 is found.

Crate:  spin
Title:  spin is no longer actively maintained
Date:   2019-11-21
URL:    https://rustsec.org/advisories/RUSTSEC-2019-0031
Dependency tree: 
spin 0.5.2
├── ring 0.16.11
│   └── crypto 0.1.0
│       └── ctap2 0.1.0
└── linked_list_allocator 0.6.6
    └── libtock 0.1.0
        ├── ctap2 0.1.0
        └── crypto 0.1.0

Next steps

  1. I filed an issue for linked_list_allocator: Spin is no longer actively maintained (RUSTSEC-2019-0031) rust-osdev/linked-list-allocator#22.
  • The libtock-rs version we're based on uses spin to lock the allocator (here), even though there is only one thread of execution so the lock isn't necessary and I don't expect much issue given that only one thread acquires the lock only once.
  • Upstream libtock-rs doesn't lock the allocator anymore (Support for RISC-V without atomic tock/libtock-rs#107), so we can pick that up as a patch in the short term.
  1. The ring dependency is only used for unit tests, and the advisory is tracked here: spin-rs no longer maintained (dependency) briansmith/ring#921.
@gendx gendx changed the title Cargo audit warns about Cargo audit warns about RUSTSEC-2019-0031 Feb 6, 2020
@gendx
Copy link
Collaborator Author

gendx commented Feb 19, 2020

The linked_list_allocator warning is now removed, due to not depending on the spin feature.

@gendx gendx mentioned this issue Jul 6, 2020
2 tasks
@kaczmarczyck
Copy link
Collaborator

After #662, cargo audit is happy.

@kaczmarczyck kaczmarczyck self-assigned this Nov 9, 2023
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

No branches or pull requests

2 participants