Skip to content

v12.22.12 proposal #42531

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.

Sign up for GitHub

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

Merged
richardlau merged 15 commits into from
Apr 5, 2022
Merged

v12.22.12 proposal #42531

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
a35f553
doc: add release key for Bryan English
bengl Feb 24, 2022
1193290
deps: V8: cherry-pick cc9a8a37445e
devsnek Mar 30, 2022
171bb66
node-api: force env shutdown deferring behavior
Feb 10, 2021
a2f4206
node-api: stop ref gc during environment teardown
gabrielschulhof Mar 4, 2021
2aa9ca1
node-api: fix crash in finalization
mhdawson Mar 23, 2021
81b4dc8
node-api: make reference weak parameter an indirect link to references
legendecas Mar 31, 2021
e707514
src: fix finalization crash
jasnell Apr 15, 2021
a7224c9
node-api: fix shutdown crashes
mhdawson Apr 30, 2021
e23c04f
node-api: avoid SecondPassCallback crash
mhdawson Jun 2, 2021
5f104e3
node-api: cctest on v8impl::Reference
legendecas Jun 8, 2021
a9c38f1
doc: add release key for Danielle Adams
danielleadams Oct 7, 2020
11aef2a
doc: update release key for Danielle Adams
danielleadams Dec 28, 2020
518a49c
doc: use openpgp.org for keyserver examples
nschonni Feb 18, 2022
333eda8
doc: add a note about possible missing lines to readline.asyncIterator
mikhalev-im Aug 8, 2020
a3d2837
2022-04-05, Version 12.22.12 'Erbium' (LTS)
richardlau Mar 30, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
src: fix finalization crash 
PR-URL: #38250
Backport-PR-URL: #42512
Fixes: #38040
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
  • Loading branch information
@jasnell @richardlau
jasnell authored and richardlau committed Mar 30, 2022
commit e707514c80371971027cd42682931f171aedd125
4 changes: 4 additions & 0 deletions src/js_native_api_v8.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -379,6 +379,9 @@ class Reference : public RefBase {

protected:
inline void Finalize(bool is_env_teardown = false) override {
if (is_env_teardown) env_teardown_finalize_started_ = true;
if (!is_env_teardown && env_teardown_finalize_started_) return;

// During env teardown, `~napi_env()` alone is responsible for finalizing.
// Thus, we don't want any stray gc passes to trigger a second call to
// `RefBase::Finalize()`. ClearWeak will ensure that even if the
Expand Down Expand Up @@ -467,6 +470,7 @@ class Reference : public RefBase {
reference->Finalize();
}

bool env_teardown_finalize_started_ = false;
v8impl::Persistent<v8::Value> _persistent;
SecondPassCallParameterRef* _secondPassParameter;
};
Expand Down