Skip to content

Respect signals-based-traps in component trampolines - #12626

Merged
fitzgen merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:component-signals-based-traps
Feb 20, 2026
Merged

Respect signals-based-traps in component trampolines#12626
fitzgen merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:component-signals-based-traps

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

This commit fixes a regression from #12547 where traps happening in component trampolines weren't respecting the signals-based-traps configuration of the Engine meaning that native CLIF traps could be executed in environments which weren't configured to catch it. This refactors the implementation internally to share more code with FuncEnvironment through some traits to ensure that the same knobs and plumbing are used to translate traps.

This commit fixes a regression from bytecodealliance#12547 where traps happening in
component trampolines weren't respecting the signals-based-traps
configuration of the `Engine` meaning that native CLIF traps could be
executed in environments which weren't configured to catch it. This
refactors the implementation internally to share more code with
`FuncEnvironment` through some traits to ensure that the same knobs and
plumbing are used to translate traps.
@alexcrichton
alexcrichton requested review from a team as code owners February 19, 2026 22:58
@alexcrichton
alexcrichton requested review from fitzgen and removed request for a team February 19, 2026 22:58
@alexcrichton alexcrichton added the fuzz-bug Bugs found by a fuzzer label Feb 19, 2026
@fitzgen
fitzgen added this pull request to the merge queue Feb 20, 2026
Merged via the queue into bytecodealliance:main with commit ee7e125 Feb 20, 2026
45 checks passed
alexcrichton added a commit to alexcrichton/wasmtime that referenced this pull request Feb 23, 2026
github-merge-queue Bot pushed a commit that referenced this pull request Feb 23, 2026
@alexcrichton
alexcrichton deleted the component-signals-based-traps branch May 17, 2026 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fuzz-bug Bugs found by a fuzzer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants