alexcrichton requested cfallin for a review on PR #13010.
alexcrichton requested wasmtime-core-reviewers for a review on PR #13010.
alexcrichton opened PR #13010 from alexcrichton:fix-winch-table-fill to bytecodealliance:main:
Note: this is a fix for GHSA-q49f-xg75-m9xw
This commit fixes a possible panic when a Winch-compiled module executes the
table.fillinstruction. Refactoring in #11254 updated Cranelift but forgot to update Winch meaning that Winch's indices were still using the module-level indices instead of theDefinedTableIndexspace. This adds some tests and updates Winch's translation to use preexisting helpers.<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #13010.
cfallin submitted PR review.
cfallin has enabled auto merge for PR #13010.
cfallin added PR #13010 winch: Fix a host panic when executing table.fill to the merge queue
cfallin merged PR #13010.
cfallin removed PR #13010 winch: Fix a host panic when executing table.fill from the merge queue
Last updated: Apr 12 2026 at 23:10 UTC