Stream: git-wasmtime

Topic: wasmtime / PR #6434 Cranelift/ABI: catch overflow in layo...

view this post on Zulip Wasmtime GitHub notifications bot (May 22 2023 at 20:54):

cfallin opened PR #6434 from cfallin:fix-stackslot-overflow to bytecodealliance:main:

This PR catches and successfully errors out of compilation a situation where stackslots exceed the u32 (4GB) range of the stack-frame layout computation.

Note that this is not reachable from Wasmtime (it does not use stackslots) so does not have security implications there, but could cause issues in other compilers (e.g. cg\_clif) that pass arbitrary stackslot sizes to Cranelift.

Fixes #6431.

view this post on Zulip Wasmtime GitHub notifications bot (May 22 2023 at 20:54):

cfallin requested abrown for a review on PR #6434.

view this post on Zulip Wasmtime GitHub notifications bot (May 22 2023 at 20:54):

cfallin requested wasmtime-compiler-reviewers for a review on PR #6434.

view this post on Zulip Wasmtime GitHub notifications bot (May 22 2023 at 21:27):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (May 23 2023 at 00:17):

cfallin merged PR #6434.

Last updated: Oct 23 2024 at 20:03 UTC