src,test: add V8 API to test the hash seed

nodejs · targos · Apr 1, 2025 · Apr 1, 2025 · Feb 26, 2025 · Sep 21, 2022
commit 6764e4a21a48cb854c11be2e46b49a1a05a73375
diff --git a/Makefile b/Makefile
@@ -654,10 +654,6 @@ test-internet: all ## Run internet tests.
 test-tick-processor: all ## Run tick processor tests.
 	$(PYTHON) tools/test.py $(PARALLEL_ARGS) tick-processor
 
-.PHONY: test-hash-seed
-test-hash-seed: all ## Verifu that the hash seed used by V8 for hashing is random.
-	$(NODE) test/pummel/test-hash-seed.js
-
 .PHONY: test-doc
 test-doc: doc-only lint-md ## Build, lint, and verify the docs.
 	@if [ "$(shell $(node_use_openssl_and_icu))" != "true" ]; then \
@@ -751,8 +747,6 @@ test-v8: v8  ## Run the V8 test suite on deps/v8.
 				mjsunit cctest debugger inspector message preparser \
 				$(TAP_V8)
 	$(call convert_to_junit,$(TAP_V8_JSON))
-	$(info Testing hash seed)
-	$(MAKE) test-hash-seed
 
 test-v8-intl: v8 ## Run the v8 test suite, intl tests.
 	export PATH="$(NO_BIN_OVERRIDE_PATH)" && \
@@ -768,7 +762,7 @@ test-v8-benchmarks: v8 ## Run the v8 test suite, benchmarks.
 				$(TAP_V8_BENCHMARKS)
 	$(call convert_to_junit,$(TAP_V8_BENCHMARKS_JSON))
 
-test-v8-updates: ## Run the v8 test suite, updates.
+test-v8-updates: all ## Run the v8 test suite, updates.
 	$(PYTHON) tools/test.py $(PARALLEL_ARGS) --mode=$(BUILDTYPE_LOWER) v8-updates
 
 test-v8-all: test-v8 test-v8-intl test-v8-benchmarks test-v8-updates ## Run the entire V8 test suite, including intl, benchmarks, and updates.

diff --git a/src/node_v8.cc b/src/node_v8.cc
@@ -32,6 +32,7 @@
 namespace node {
 namespace v8_utils {
 using v8::Array;
+using v8::BigInt;
 using v8::CFunction;
 using v8::Context;
 using v8::FunctionCallbackInfo;
@@ -260,6 +261,12 @@ static bool FastIsStringOneByteRepresentation(Local<Value> receiver,
 CFunction fast_is_string_one_byte_representation_(
     CFunction::Make(FastIsStringOneByteRepresentation));
 
+void GetHashSeed(const FunctionCallbackInfo<Value>& args) {
+  Isolate* isolate = args.GetIsolate();
+  uint64_t hash_seed = isolate->GetHashSeed();
+  args.GetReturnValue().Set(BigInt::NewFromUnsigned(isolate, hash_seed));
+}
+
 static const char* GetGCTypeName(v8::GCType gc_type) {
   switch (gc_type) {
     case v8::GCType::kGCTypeScavenge:
@@ -694,6 +701,8 @@ void Initialize(Local<Object> target,
                             IsStringOneByteRepresentation,
                             &fast_is_string_one_byte_representation_);
 
+  SetMethodNoSideEffect(context, target, "getHashSeed", GetHashSeed);
+
   // GCProfiler
   Local<FunctionTemplate> t =
       NewFunctionTemplate(env->isolate(), GCProfiler::New);
@@ -721,6 +730,7 @@ void RegisterExternalReferences(ExternalReferenceRegistry* registry) {
   registry->Register(UpdateHeapCodeStatisticsBuffer);
   registry->Register(UpdateHeapSpaceStatisticsBuffer);
   registry->Register(SetFlagsFromString);
+  registry->Register(GetHashSeed);
   registry->Register(SetHeapSnapshotNearHeapLimit);
   registry->Register(GCProfiler::New);
   registry->Register(GCProfiler::Start);

diff --git a/test/fixtures/guess-hash-seed.js b/test/fixtures/guess-hash-seed.js
diff --git a/test/parallel/test-hash-seed.mjs b/test/parallel/test-hash-seed.mjs
@@ -0,0 +1,27 @@
+import '../common/index.mjs';
+
+import assert from 'node:assert';
+import { execFile } from 'node:child_process';
+import { promisify, debuglog } from 'node:util';
+
+// This test verifies that the V8 hash seed is random
+// and unique between child processes.
+
+const execFilePromise = promisify(execFile);
+const debug = debuglog('test');
+
+const kRepetitions = 3;
+
+const seeds = await Promise.all(Array.from({ length: kRepetitions }, generateSeed));
+debug(`Seeds: ${seeds}`);
+assert.strictEqual(new Set(seeds).size, seeds.length);
+assert.strictEqual(seeds.length, kRepetitions);
+
+async function generateSeed() {
+  const output = await execFilePromise(process.execPath, [
+    '--expose-internals',
+    '--print',
+    'require("internal/test/binding").internalBinding("v8").getHashSeed()',
+  ]);
+  return output.stdout.trim();
+}
diff --git a/test/pummel/test-hash-seed.js b/test/pummel/test-hash-seed.js