-
-
Notifications
You must be signed in to change notification settings - Fork 293
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
added write_file_atomic against ondisk corpus races #294
Conversation
Perhaps it would make sense to have a |
Right now the code is more or less copy and pasted so definitely makes sense. Where should that go? Bolts? |
I'd imagine bolts file utils or something? |
Added |
Awesome! Will test this once it is merged. |
* fix ondisk corpus race condition * move metadata name to be a dotfile * note ExitKind for crashes and timeouts in inprocess executor * potential fix for windows * added write_file_atomic * no_std fixes * no_std testcase fix * typo fix, windows * clippy * more no_std testing
This PR fixes #289 by atomically adding a lockfile before using a filename.
It appends a number to the filename in case of a collision.
Also, it first writes to a tmp file, then moves to the actual filename.
Each of them are dofiles (which should then be ignored for further processing)