Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sanitize wasm vm addresses #2385

Merged
merged 5 commits into from
Jan 13, 2024
Merged

Sanitize wasm vm addresses #2385

merged 5 commits into from
Jan 13, 2024

Conversation

sug0
Copy link
Collaborator

@sug0 sug0 commented Jan 12, 2024

Indicate on which release or other PRs this topic is based on

#2384

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

The new `wasmer` revision points to `tiago/more-alignment-fixes`, on
`heliaxdev/wasmer`.
@sug0 sug0 added the bug Something isn't working label Jan 12, 2024
@sug0 sug0 force-pushed the tiago/wasm-vm-addr-sanitize branch from 8a6e5fa to bfff151 Compare January 12, 2024 16:17
@sug0 sug0 marked this pull request as ready for review January 12, 2024 16:18
@sug0 sug0 requested a review from tzemanovic January 12, 2024 16:29
@tzemanovic tzemanovic mentioned this pull request Jan 12, 2024
tzemanovic added a commit that referenced this pull request Jan 12, 2024
* origin/tiago/wasm-vm-addr-sanitize:
  Changelog for #2385
  Add wasm ptr overflow checks regression test
  Sanitize tx and vp wasm memory accesses
@brentstone brentstone merged commit 78db6d9 into main Jan 13, 2024
12 of 15 checks passed
@brentstone brentstone deleted the tiago/wasm-vm-addr-sanitize branch January 13, 2024 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants