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

[16.0.0] Backport adapter_{open|close}_badfd exports #7675

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

dicej
Copy link
Contributor

@dicej dicej commented Dec 12, 2023

This backports #7663 and #7672 to the release-16.0.0 branch, which will make it easier for us to test the ongoing work to support wasi-sockets in wasi-libc.

dicej and others added 2 commits December 12, 2023 07:25
…dealliance#7663)

* add `adapter_{open|close}_badfd` exports to Preview 1 adapter

This is to be used by `wasi-libc` to reserve file descriptors for its own use
(e.g. for sockets), ensuring that any attempt to pass them directly to Preview 1
functions will consistently return an error.

See WebAssembly/wasi-libc#447 for further details.

Signed-off-by: Joel Dice <joel.dice@fermyon.com>

* add `preview2_adapter_badfd` test

Signed-off-by: Joel Dice <joel.dice@fermyon.com>

---------

Signed-off-by: Joel Dice <joel.dice@fermyon.com>
…lliance#7672)

Makes them a bit more consistent with the rest of the WASI functions as
opposed to returning a boolean.
@dicej dicej requested a review from a team as a code owner December 12, 2023 14:34
@dicej dicej requested review from alexcrichton and removed request for a team December 12, 2023 14:34
@dicej dicej changed the title [16.0.0] Backport adapter_{open|close}_badf exports [16.0.0] Backport adapter_{open|close}_badfd exports Dec 12, 2023
@alexcrichton alexcrichton merged commit 64c2a24 into bytecodealliance:release-16.0.0 Dec 12, 2023
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants