Skip to content

Fix free_iovs guest pointer mapping in polyfill#153

Merged
sunfishcode merged 1 commit intobytecodealliance:masterfrom
willscott:polyfill-iovs
May 18, 2019
Merged

Fix free_iovs guest pointer mapping in polyfill#153
sunfishcode merged 1 commit intobytecodealliance:masterfrom
willscott:polyfill-iovs

Conversation

@willscott
Copy link

I needed this change in order for data from stdin to be correctly received in the guest wasi module buffer when running in the polyfill.

@sunfishcode
Copy link
Member

Oops, good find! Running in the simple shell.html driver, I hadn't yet had a chance to test stdin.

I'll add a README about this, but just so you know, the polyfill is now being maintained on the polyfill branch, so I'll cherry-pick this PR over there.

@sunfishcode sunfishcode merged commit 1320b11 into bytecodealliance:master May 18, 2019
grishasobol pushed a commit to grishasobol/wasmtime that referenced this pull request Nov 29, 2021
* Track memory usage

* Track lowest_used in copy, etc.

* Extra comment for used_size
pchickey added a commit to pchickey/wasmtime that referenced this pull request May 16, 2023
* Revert "wasi-tests: add configuration to ignore rights readback (bytecodealliance#81)"

This reverts commit d006b1b.

* wasi-tests: pull in all changes from upstream which remove support for rights

bytecodealliance#6265

note, im leaving out upstream changes to fd_readdir which are from bytecodealliance#6163, which requires porting the rest of that PR into this repo

* remove rights readback variable from test runner

* rustfmt

* delete path_open_read_without_rights test

which was deleted upstream
mooori pushed a commit to mooori/wasmtime that referenced this pull request Dec 20, 2023
dhil added a commit to dhil/wasmtime that referenced this pull request Apr 4, 2024
This patch improves the local names used within the `call_builtin` ever-so-slightly.
dicej added a commit to dicej/wasmtime that referenced this pull request May 8, 2025
…utures-and-streams-rs-unsafe-comments

add comments to unsafe code in `futures_and_streams.rs`
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