Skip to content

Set SOCK_CLOEXEC on Accept #400

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.

Sign up for GitHub

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

Jump to bottom
Closed
Berrysoft opened this issue May 7, 2025 · 0 comments · Fixed by #403
Closed

Set SOCK_CLOEXEC on Accept #400

Berrysoft opened this issue May 7, 2025 · 0 comments · Fixed by #403
Assignees
@Berrysoft
Labels
bug Something isn't working driver: fusion About the fusion driver driver: io-uring About the io-uring driver driver: IOCP About the IOCP driver driver: polling About the polling driver

Comments

@Berrysoft
Copy link
Member

For polling backend, use accept4 as long as possible.

For IOCP backend, ensure WSA_FLAG_NO_HANDLE_INHERIT.
@Berrysoft Berrysoft self-assigned this May 7, 2025
@Berrysoft Berrysoft added bug Something isn't working driver: IOCP About the IOCP driver driver: io-uring About the io-uring driver driver: polling About the polling driver driver: fusion About the fusion driver labels May 7, 2025
@Berrysoft Berrysoft mentioned this issue May 10, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Berrysoft Berrysoft

Labels
bug Something isn't working driver: fusion About the fusion driver driver: io-uring About the io-uring driver driver: IOCP About the IOCP driver driver: polling About the polling driver
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add CLOEXEC flags Berrysoft/compio
1 participant
@Berrysoft