alexcrichton opened PR #14922 from alexcrichton:component-deserialize-oom to bytecodealliance:main:
This commit updates various data structures within a
Componentto reflect that OOM must be handled and then cascades the change throughout. This ensures that components can be deserialized while handling OOM and returning it to the caller.<!--
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 review the Bytecode Alliance's AI tool usage policy at
https://github.com/bytecodealliance/governance/blob/main/AI_TOOL_POLICY.mdPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton removed the review request for wasmtime-fuzz-reviewers from PR #14922.
alexcrichton requested wasmtime-fuzz-reviewers for a review on PR #14922.
alexcrichton requested wasmtime-core-reviewers for a review on PR #14922.
alexcrichton requested fitzgen for a review on PR #14922.
github-actions[bot] added the label wasmtime:api on PR #14922.
github-actions[bot] added the label fuzzing on PR #14922.
github-actions[bot] commented on PR #14922:
Subscribe to Label Action
cc @fitzgen
<details>
This issue or pull request has been labeled: "fuzzing", "wasmtime:api"Thus the following users have been cc'd because of the following labels:
- fitzgen: fuzzing
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
:thumbs_up: fitzgen submitted PR review.
:speech_balloon: fitzgen created PR review comment:
Are we missing a
Deref<str> for TryStringor something?
:memo: alexcrichton submitted PR review.
:speech_balloon: alexcrichton created PR review comment:
Ah the error here is:
the trait `core::borrow::Borrow<std::string::String>` is not implemented for `wasmtime_internal_core::alloc::TryString`So more just a
String/TryStringincompatibility
alexcrichton added PR #14922 Handle OOM in Component::deserialize to the merge queue
:check: alexcrichton merged PR #14922.
alexcrichton removed PR #14922 Handle OOM in Component::deserialize from the merge queue
Last updated: Oct 31 2026 at 23:03 UTC