Skip to content

node-api: fix shutdown crashes#38492

Closed
mhdawson wants to merge 6 commits intonodejs:masterfrom
mhdawson:node-api-crash-fix
Closed

node-api: fix shutdown crashes#38492
mhdawson wants to merge 6 commits intonodejs:masterfrom
mhdawson:node-api-crash-fix

Conversation

@mhdawson
Copy link
Copy Markdown
Member

Refs: nodejs/node-addon-api#906

Ensure that finalization is not defered during shutdown.
The env for the addon is deleted immediately after
iterating the list of finalizers to be run. Defering
causes crashes as the finalization uses the already
deleted env.

Signed-off-by: Michael Dawson [email protected]

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. node-api Issues and PRs related to the Node-API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants