src: readiterable entries may be empty

fixup

PR-URL: #50398
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Debadree Chatterjee <[email protected]>

Author: KhafraDev

src/node_messaging.cc

@@ -1003,8 +1003,11 @@ static Maybe<bool> ReadIterable(Environment* env,
     entries.push_back(val);
   }
 
-  transfer_list.AllocateSufficientStorage(entries.size());
-  std::copy(entries.begin(), entries.end(), &transfer_list[0]);
+  if (!entries.empty()) {
+    transfer_list.AllocateSufficientStorage(entries.size());
+    std::copy(entries.begin(), entries.end(), &transfer_list[0]);
+  }
+
   return Just(true);
 }
 

test/parallel/test-messagechannel.js

@@ -0,0 +1,12 @@
+'use strict';
+
+const common = require('../common');
+
+// See: https://github.com/nodejs/node/issues/49940
+(async () => {
+  new MessageChannel().port1.postMessage({}, {
+    transfer: {
+      *[Symbol.iterator]() {}
+    }
+  });
+})().then(common.mustCall());

test/parallel/test-structuredClone-global.js

@@ -15,3 +15,14 @@ assert.strictEqual(structuredClone(undefined, null), undefined);
 // Transfer can be null or undefined.
 assert.strictEqual(structuredClone(undefined, { transfer: null }), undefined);
 assert.strictEqual(structuredClone(undefined, { }), undefined);
+
+{
+  // See: https://github.com/nodejs/node/issues/49940
+  const cloned = structuredClone({}, {
+    transfer: {
+      *[Symbol.iterator]() {}
+    }
+  });
+
+  assert.deepStrictEqual(cloned, {});
+}