fix fopen() overwriting stdin file handle #3
Conversation
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
See adtools#2 Fix is extracted from the fork by @bebbo, specifically the two following commits: https://github.com/bebbo/libnix/commit/fa82b7a1aa5ca575d79ff8d4b5c55da24b19f835 https://github.com/bebbo/libnix/commit/95dbee7b50409bad6bae725c8f5eab0c836a2338
Author
|
PING? |
from the fork by @bebbo, specifically the two following commits: https://github.com/bebbo/libnix/commit/480d298f6e109b6f2813a22b06aef30cfe9ef786 https://github.com/bebbo/libnix/commit/b3b28a22251f5e4ac0f9d9b505f13ea2e27d1a08
from the fork by @bebbo, specifically the following commit: https://github.com/bebbo/libnix/commit/480d298f6e109b6f2813a22b06aef30cfe9ef786
from the fork by @bebbo, specifically the following commit: https://github.com/bebbo/libnix/commit/1e56fc31d3273e5ee402e68aa9843b94799f2859
bebbo's toolchain does it differently, same end result.
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.
See #2
Fix is extracted from the fork by @bebbo, specifically the two following commits:
https://github.com/bebbo/libnix/commit/fa82b7a1aa5ca575d79ff8d4b5c55da24b19f835
https://github.com/bebbo/libnix/commit/95dbee7b50409bad6bae725c8f5eab0c836a2338
Please review and merge if it's OK.