vm: eval cache mixes up script origins

Offen

#65,866 erstellt am 06.9.2026 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Aktiv
Tech-Stack: cpp, javascript, node.js
Bereich: backend

Rechercherichtung

Reproduziere das Problem mit der bereitgestellten test.mjs unter --experimental-vm-modules und untersuche anschließend deps/v8/src/codegen/compiler.cc rund um den Kommentar "cache lookup key". Verfolge die Suche im eval-Cache und die zugehörigen Tests, um zu verstehen, wie Skriptherkünfte dargestellt werden. Erledigt ist die Aufgabe, wenn die A/B/A-Assertion bei aktiviertem Komilierungs-Cache besteht und sich das bestehende Verhalten ohne Kompilierungs-Cache weiterhin korrekt verhält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Bug report

Node.js: v24.19.0, v26.8.1, v27.0.0-pre (local build)
Platform: Microsoft Windows NT 10.0.26200.0 x64
Subsystem: vm

// test.mjs
import assert from 'node:assert/strict';
import { Script } from 'node:vm';

const out = [];

function createScript(name) {
  return new Script(`globalThis.eval('import("node:fs")')`, {
    importModuleDynamically(specifier) {
      out.push(name);
      return import(specifier);
    },
  });
}

const a = createScript('A');
const b = createScript('B');

for (const script of [a, b, a]) {
  await script.runInThisContext();
}

assert.deepEqual(out, ['A', 'B', 'A']);

Running this example as node --experimental-vm-modules test.mjs will give different wrong answers between v24 and v26 because V8 removed delayed caching some time during those releases. However, running with --no-compilation-cache will pass the test assertion.

The bug is with V8. In deps/v8/src/codegen/compiler.cc, around the "cache lookup key" comment, indirect eval uses kNoSourcePosition (-1), and the cache key does not distinguish the originating script. This allows identical eval source text from different scripts in the same realm to produce a false positive cache hit.

Vorherrschende Sprache: JavaScript
Sterne: 122k
Forks: 37.4k
Ø Merge: 4 T. 3 Std.
Gemergte PRs (30 T.): 279

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus nodejs/node

Stream.prototype.forEach will block in first promise in queue before read more chunk Offen

doc

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

nodejs/node#66106 · Erstellt am 18.9.2026
nodejs 26.8.2 fails to build with clang/LLVM Offen

build

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100

nodejs/node#66076 · 2 Kommentare · Erstellt am 17.9.2026
`TextEncoder.encodeInto()` underfills the destination for some non-ASCII text Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100

nodejs/node#65994 · 2 Kommentare · 2 Reaktionen · Erstellt am 11.9.2026
worker: `messageerror` listeners remain during `exit` Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100

nodejs/node#65782 · Erstellt am 04.9.2026
[`util.styleText`] Allow `text` param to be any scalar value (ex. `number`, `bool` etc additionaly to `string`) which will be casted internally to `string` Offen

feature request

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

nodejs/node#63841 · 1 Kommentar · Erstellt am 10.6.2026

Alle Issues in nodejs/node