"pure virtual method called" when creating a heap snapshot

[laino]

• Version: v11.15.0
• Platform: Linux

Run the following script with node --inspect script.js, connect a debugger with Google Chrome and instruct it to do a heap snapshot.

Edit: See my comment below for a better snippet.

function memoryLeak() {
 return new Promise(() => {});
}

function loop() {
 for (let i = 0; i < 10000; i++) {
 memoryLeak();
 }

 setTimeout(loop, 0);
}

loop();

Roughly half of the time node will crash with "pure virtual method called".

That code actually causes a memory leak, but it will likely be annoying to debug if this isn't fixed first.

[laino]

This code crashes 100% of the time instead of 50% of the time:

function memoryLeak() {
 return new Promise(() => {});
}

async function loop() {
 for (let i = 0; i < 10000; i++) {
 memoryLeak();
 }
}

async function run() {
 while (true) {
 await loop();
 }
}

run();

[addaleax]

Thanks for the excellent bug report! #28789 should address this.