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

limit the amount of data recorded from libFuzzer's stderr #785

Merged
merged 5 commits into from
Apr 9, 2021

Conversation

bmc-msft
Copy link
Contributor

@bmc-msft bmc-msft commented Apr 9, 2021

Upon failure, OneFuzz reports the stderr as part of the task failure. In some cases, this could end up being an excessive amount of data kept in memory. This limits the stderr saved in case of failure to at most the last 1024 lines.

Upon failure, OneFuzz reports the stderr as part of the task failure.  In some cases, this could end up being an excessive amount of data kept in memory.  This limits the stderr saved in case of failure to at most the last 1000 lines.
@bmc-msft bmc-msft marked this pull request as draft April 9, 2021 14:33
@bmc-msft bmc-msft marked this pull request as ready for review April 9, 2021 14:51
@ghost
Copy link

ghost commented Apr 9, 2021

Hello @bmc-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 18 hours, a condition that will be fulfilled in about 16 hours 16 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@bmc-msft bmc-msft merged commit 030ab4b into microsoft:main Apr 9, 2021
@bmc-msft bmc-msft deleted the limit-libfuzzer-stderr-kept branch April 9, 2021 19:45
@ghost ghost locked as resolved and limited conversation to collaborators May 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants