<!-- Thank you for reporting an issue. This issue tracker is for bugs and issues found within Node.js core. If you require more general support please file an issue on our help repo. https://github.com/nodejs/help Please fill in as much of the template below as you're able. Version: output of `node -v` Platform: output of `uname -a` (UNIX), or version and 32 or 64-bit (Windows) Subsystem: if known, please specify affected core module name --> * **Version**: 12.x and higher * **Platform**: ALL * **Subsystem**: N-API Refs: https://github.com/nodejs/node/pull/35777 ### What steps will reproduce the bug? Run test test/node-api/test_worker_terminate_finalization/test.js under ASAN/memory check ### How often does it reproduce? Is there a required condition? ### What is the expected behavior? No memory leaks reported ### What do you see instead? Memory leaks reported ### Additional information We need to investigate the test to see if it is an issue in the test or in N-API itself.
Activity
mhdawson commentedon Oct 23, 2020
Test is currently excluded by:
// TODO(addaleax): Run this test once it stops failing under ASAN/valgrind.
// Refs: #34731
common.skip('Reference management in N-API leaks memory');
n-api: unlink reference during its destructor
n-api: unlink reference during its destructor
n-api: unlink reference during its destructor
n-api: unlink reference during its destructor
n-api: unlink reference during its destructor