Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

fixing libfuzzer_merge #240

Merged
merged 10 commits into from
Nov 3, 2020
Merged

fixing libfuzzer_merge #240

merged 10 commits into from
Nov 3, 2020

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Oct 30, 2020

Summary of the Pull Request

Fix the the libfuzzer_merge tasks
Include libfuzzer_merge tasks in the libfuzzer template
Update future crate to 0.3.7 because of a vulnerability reported by cargo audit

PR Checklist

  • Applies to work item: libfuzzer_merge is broken: expected struct SyncedDir #235
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

run lib_fuzzer remplate

onefuzz template libfuzzer  basic test test 1 pool_linux

@chkeita chkeita changed the title Chkeita/235 fixing libfuzzer_merge Oct 30, 2020
@chkeita chkeita linked an issue Oct 30, 2020 that may be closed by this pull request
@@ -16,7 +16,7 @@ async-trait = "0.1"
clap = "2.33"
tempfile = "3.1"
env_logger = "0.7"
futures = "0.3"
futures = "0.3.7"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated because of an issue reported by cargo audit:

Crate:         futures-util
Version:       0.3.6
Title:         MutexGuard::map can cause a data race in safe code
Date:          2020-10-22
ID:            RUSTSEC-2020-0059
URL:           https://rustsec.org/advisories/RUSTSEC-2020-0059
Solution:      Upgrade to >=0.3.7

Copy link
Contributor

Choose a reason for hiding this comment

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

I had to merge this change as part of the cargo-audit fix #242.

@chkeita chkeita marked this pull request as ready for review October 31, 2020 01:48
src/cli/onefuzz/templates/libfuzzer.py Outdated Show resolved Hide resolved
@@ -16,7 +16,7 @@ async-trait = "0.1"
clap = "2.33"
tempfile = "3.1"
env_logger = "0.7"
futures = "0.3"
futures = "0.3.7"
Copy link
Contributor

Choose a reason for hiding this comment

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

I had to merge this change as part of the cargo-audit fix #242.

@bmc-msft bmc-msft merged commit 04643a9 into microsoft:main Nov 3, 2020
@chkeita chkeita deleted the chkeita/235 branch March 31, 2021 21:52
@ghost ghost locked as resolved and limited conversation to collaborators May 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

libfuzzer_merge is broken: expected struct SyncedDir
2 participants