nodejs
diff --git a/Collapse file
‎src/node_messaging.cc‎
Copy file name to clipboardExpand all lines: src/node_messaging.cc
+7-2Lines changed: 7 additions & 2 deletions b/Collapse file
‎src/node_messaging.cc‎
Copy file name to clipboardExpand all lines: src/node_messaging.cc
+7-2Lines changed: 7 additions & 2 deletions
diff --git a/Collapse file
‎test/parallel/test-whatwg-webstreams-transform-stream-members.js‎
Copy file name to clipboard
+19Lines changed: 19 additions & 0 deletions b/Collapse file
‎test/parallel/test-whatwg-webstreams-transform-stream-members.js‎
Copy file name to clipboard
+19Lines changed: 19 additions & 0 deletions
diff --git a/Collapse file
‎test/wpt/status/streams.json‎
Copy file name to clipboardExpand all lines: test/wpt/status/streams.json
-8Lines changed: 0 additions & 8 deletions b/Collapse file
‎test/wpt/status/streams.json‎
Copy file name to clipboardExpand all lines: test/wpt/status/streams.json
-8Lines changed: 0 additions & 8 deletions
@@ -415,10 +415,15 @@ class SerializerDelegate : public ValueSerializer::Delegate {
       if (!host_objects_[i]->NestedTransferables().To(&nested_transferables))
         return Nothing<bool>();
       for (auto& nested_transferable : nested_transferables) {
-        if (std::ranges::find(host_objects_, nested_transferable) ==
+        if (std::ranges::find(host_objects_, nested_transferable) !=
             host_objects_.end()) {
-          AddHostObject(nested_transferable);
+          ThrowDataCloneException(
+              context_,
+              FIXED_ONE_BYTE_STRING(env_->isolate(),
+                                    "The transfer list is invalid."));
+          return Nothing<bool>();
         }
+        AddHostObject(nested_transferable);
       }
     }
     return Just(true);
 
@@ -0,0 +1,19 @@
+'use strict';
+
+require('../common');
+const assert = require('node:assert');
+
+const combinations = [
+  ((t) => [t, t.readable])(new TransformStream()),
+  ((t) => [t.readable, t])(new TransformStream()),
+  ((t) => [t, t.writable])(new TransformStream()),
+  ((t) => [t.writable, t])(new TransformStream()),
+];
+
+for (const combination of combinations) {
+  assert.throws(() => structuredClone(combination, { transfer: combination }), {
+    constructor: DOMException,
+    name: 'DataCloneError',
+    code: 25,
+  });
+}
@@ -41,14 +41,6 @@
   "transferable/transfer-with-messageport.window.js": {
     "skip": "Browser-specific test"
   },
-  "transferable/transform-stream-members.any.js": {
-    "fail": {
-      "expected": [
-        "Transferring [object TransformStream],[object ReadableStream] should fail",
-        "Transferring [object TransformStream],[object WritableStream] should fail"
-      ]
-    }
-  },
   "transform-streams/invalid-realm.tentative.window.js": {
     "skip": "Browser-specific test"
   }
Original file line number	Diff line number	Diff line change
`@@ -415,10 +415,15 @@ class SerializerDelegate : public ValueSerializer::Delegate {`
`415`	`415`	`if (!host_objects_[i]->NestedTransferables().To(&nested_transferables))`
`416`	`416`	`return Nothing<bool>();`
`417`	`417`	`for (auto& nested_transferable : nested_transferables) {`
`418`		`- if (std::ranges::find(host_objects_, nested_transferable) ==`
	`418`	`+ if (std::ranges::find(host_objects_, nested_transferable) !=`
`419`	`419`	`host_objects_.end()) {`
`420`		`- AddHostObject(nested_transferable);`
	`420`	`+ ThrowDataCloneException(`
	`421`	`+ context_,`
	`422`	`+ FIXED_ONE_BYTE_STRING(env_->isolate(),`
	`423`	`+ "The transfer list is invalid."));`
	`424`	`+ return Nothing<bool>();`
`421`	`425`	`}`
	`426`	`+ AddHostObject(nested_transferable);`
`422`	`427`	`}`
`423`	`428`	`}`
`424`	`429`	`return Just(true);`