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

wasi: fix serdes bugs from snapshot1 migration #31122

Closed
wants to merge 1 commit into from

Commits on Dec 28, 2019

  1. wasi: fix serdes bugs from snapshot1 migration

    During the migration to WASI snapshot1, a field was removed
    from the subscription type. The field was removed from the
    code, but the bounds checking logic was not updated. This
    commit updates that check.
    
    Similarly, __wasi_linkcount_t changed from a uint32_t to a
    uint64_t. However, the bounds checks were missed, and the code
    was still writing uint32_t's (to the new correct offset) instead
    of uint64_t's. This commit updates that logic as well.
    
    Refs: nodejs#30980
    cjihrig committed Dec 28, 2019
    Configuration menu
    Copy the full SHA
    10bd3b7 View commit details
    Browse the repository at this point in the history