diff options
| author | Anna Henningsen <anna@addaleax.net> | 2019-07-20 23:12:41 +0200 |
|---|---|---|
| committer | Rich Trott <rtrott@gmail.com> | 2019-07-22 16:41:22 -0700 |
| commit | 9949fbda469e70acc45975f5e50eedd2a3cde57d (patch) | |
| tree | abdf778a51515a387c1b6ba6087017dc103e6fa3 /tools | |
| parent | 89e4b36e62978f54b2e33b4bce8197072dbe8af1 (diff) | |
| download | android-node-v8-9949fbda469e70acc45975f5e50eedd2a3cde57d.tar.gz android-node-v8-9949fbda469e70acc45975f5e50eedd2a3cde57d.tar.bz2 android-node-v8-9949fbda469e70acc45975f5e50eedd2a3cde57d.zip | |
src: do not include partial AsyncWrap instances in heap dump
Heap dumps can be taken either through the inspector or the public API
for it during an async_hooks init() hook, but at that point the
AsyncWrap in question is not done initializing yet and virtual methods
cannot be called on it.
Address this issue (somewhat hackily) by excluding `AsyncWrap`
instances which have not yet executed their `init()` hook fully
from heap dumps.
Fixes: https://github.com/nodejs/node/issues/28786
PR-URL: https://github.com/nodejs/node/pull/28789
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions