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: add a make task security-audit to audit Cargo.lock #38

Merged

Conversation

rainchen
Copy link
Contributor

↪  make security-audit                                                                                                                                Wed Nov 28 19:21:09 CST 2018
cargo-audit 0.5.2
    Fetching advisory database from `https://github.com/RustSec/advisory-db.git`
      Loaded 14 security advisories (from /Users/rain/.cargo/advisory-db)
    Scanning Cargo.lock for vulnerabilities (353 crate dependencies)
error: Vulnerable crates found!

ID:	 RUSTSEC-2018-0003
Crate:	 smallvec
Version: 0.2.1
Date:	 2018-07-19
URL:	 https://github.com/servo/rust-smallvec/issues/96
Title:	 Possible double free during unwinding in SmallVec::insert_many
Solution: upgrade to: >= 0.6.3 OR ^0.3.4 OR ^0.4.5 OR ^0.5.1

ID:	 RUSTSEC-2018-0001
Crate:	 untrusted
Version: 0.5.1
Date:	 2018-06-21
URL:	 https://github.com/briansmith/untrusted/pull/20
Title:	 An integer underflow could lead to panic
Solution: upgrade to: >= 0.6.2

ID:	 RUSTSEC-2018-0006
Crate:	 yaml-rust
Version: 0.3.5
Date:	 2018-09-17
URL:	 https://github.com/chyh1990/yaml-rust/pull/109
Title:	 Uncontrolled recursion leads to abort in deserialization
Solution: upgrade to: >= 0.4.1

error: 3 vulnerabilities found!
make: *** [security-audit] Error 1

We can use this task in CI job , it will exit with status 1 if there is any vulnerable crate found.

@doitian doitian self-requested a review November 28, 2018 15:38
@doitian doitian merged commit 1ca3090 into nervosnetwork:develop Nov 28, 2018
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.

None yet

2 participants