Skip to content
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

osproc: fix double close on POSIX #5724

Merged
merged 1 commit into from
Apr 16, 2018
Merged

Conversation

zielmicha
Copy link
Contributor

Calling close() in some cases issued two close() syscalls to one FD, which is incorrect in multithreaded programs.

@Araq
Copy link
Member

Araq commented Apr 23, 2017

This needs a test.

Calling close() in some cases issued two close() syscalls to one FD, which is incorrect in multithreaded programs.
@zielmicha
Copy link
Contributor Author

Test added. It's not ideal (it passed before), but at least shows there is no regression.

@dom96
Copy link
Contributor

dom96 commented Jun 13, 2017

Any chance we could get a test that reproduces the issue that this PR fixes?

@zielmicha
Copy link
Contributor Author

I can write test for this with dependency on strace.

@dom96
Copy link
Contributor

dom96 commented Jun 14, 2017

That should be fine. Make it run on Linux only.

@Araq Araq merged commit b2060ac into nim-lang:devel Apr 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants