Fix read_all() and write_all() to always set errno on failure#86
Open
vasild wants to merge 2 commits into
Open
Fix read_all() and write_all() to always set errno on failure#86vasild wants to merge 2 commits into
vasild wants to merge 2 commits into
Conversation
read(2) will return 0 to designate end-of-file and in this case it will not set errno, but read_all() will return false, leaving errno as it were before the call. An EOF is not an error for read(2) itself, but read_all() is supposed to read exactly the requested number of bytes (not less), so set errno to EBADMSG "Bad message" in this case.
write(2) could return 0 (no bytes written), but that is not an error condition, write(2) will not set errno, and the operation should be retried. This patch, in addition to making a retry on 0, also ensures that errno is always set whenever false is returned from write_all().
Author
|
About the failing travis tests: I can reproduce some hangs (maybe the same as in travis, maybe not) even without this patch, using 39b46a0 (latest or |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Make sure
errnois always set iffalseis to be returned. Also retry ifwrite(2)returns 0.