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

Fix race condition in kqueue event system implementation #2193

Merged
merged 4 commits into from Aug 26, 2017

Conversation

SeanTAllen
Copy link
Member

@SeanTAllen SeanTAllen commented Aug 25, 2017

This fixes a race condition in the kqueue back-end. Previously, it was possible for an unsubscribed (i.e. ASIO_DISPOSABLE) to be sent as an event in between being disposed and being sent as an ASIO_DISPOSABLE event. This could cause an event that had been unsubscribed to receive a read or a write event immediately before a dispose event.

@SeanTAllen SeanTAllen added the do not merge This PR should not be merged at this time label Aug 25, 2017
@SeanTAllen SeanTAllen force-pushed the signal_hunt branch 3 times, most recently from dd326a6 to 77c6df3 Compare August 25, 2017 20:26
@SeanTAllen SeanTAllen added the changelog - fixed Automatically add "Fixed" CHANGELOG entry on merge label Aug 26, 2017
@SeanTAllen SeanTAllen changed the title DO NOT MERGE Fix race condition in kqueue event system implementation Aug 26, 2017
@ponylang ponylang deleted a comment from sylvanc Aug 26, 2017
@SeanTAllen SeanTAllen merged commit 5f29a0f into master Aug 26, 2017
@SeanTAllen SeanTAllen deleted the signal_hunt branch August 26, 2017 17:02
ponylang-main added a commit that referenced this pull request Aug 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog - fixed Automatically add "Fixed" CHANGELOG entry on merge do not merge This PR should not be merged at this time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants