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: throw on failed uvwasi_init() #31076

Merged
merged 3 commits into from
Dec 26, 2019
Merged

wasi: throw on failed uvwasi_init() #31076

merged 3 commits into from
Dec 26, 2019

Commits on Dec 26, 2019

  1. deps: uvwasi: cherry-pick 64e59d5

    Original commit message:
    
        This commit ensures that multiple calls to uvwasi_destroy()
        are possible. Prior to this commit, subsequent calls to
        destroy() would abort because the file table's rwlock was
        incorrectly being destroyed multiple times.
    
    PR-URL: nodejs#31076
    Reviewed-By: Jiawen Geng <[email protected]>
    Reviewed-By: Anna Henningsen <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    Reviewed-By: Chengzhong Wu <[email protected]>
    Reviewed-By: Rich Trott <[email protected]>
    cjihrig committed Dec 26, 2019
    Configuration menu
    Copy the full SHA
    a512bf2 View commit details
    Browse the repository at this point in the history
  2. deps: uvwasi: cherry-pick 75b389c

    Original commit message:
    
        This commit changes the memory management in
        uvwasi_fd_table_init() such that ENOMEM errors can be
        handled better in uvwasi_fd_table_free().
    
    PR-URL: nodejs#31076
    Reviewed-By: Jiawen Geng <[email protected]>
    Reviewed-By: Anna Henningsen <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    Reviewed-By: Chengzhong Wu <[email protected]>
    Reviewed-By: Rich Trott <[email protected]>
    cjihrig committed Dec 26, 2019
    Configuration menu
    Copy the full SHA
    6e20674 View commit details
    Browse the repository at this point in the history
  3. wasi: throw on failed uvwasi_init()

    Prior to this commit, if uvwasi_init() failed in any way, Node
    would abort due to a failed CHECK_EQ(). This commit changes
    that behavior to a thrown exception.
    
    PR-URL: nodejs#31076
    Fixes: nodejs#30878
    Reviewed-By: Jiawen Geng <[email protected]>
    Reviewed-By: Anna Henningsen <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    Reviewed-By: Chengzhong Wu <[email protected]>
    Reviewed-By: Rich Trott <[email protected]>
    cjihrig committed Dec 26, 2019
    Configuration menu
    Copy the full SHA
    208453e View commit details
    Browse the repository at this point in the history