src: iterate on import attributes array correctly

The array's length is supposed to be a multiple of two for dynamic
import callbacks.
Fixes: #50700
PR-URL: #50703
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Shelley Vohr <shelley.vohr@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

Author: targos

src/module_wrap.cc

@@ -255,10 +255,14 @@ void ModuleWrap::New(const FunctionCallbackInfo<Value>& args) {
 }
 
 static Local<Object> createImportAttributesContainer(
- Realm* realm, Isolate* isolate, Local<FixedArray> raw_attributes) {
+ Realm* realm,
+ Isolate* isolate,
+ Local<FixedArray> raw_attributes,
+constint elements_per_attribute) {
+CHECK_EQ(raw_attributes->Length() % elements_per_attribute, 0);
  Local<Object> attributes =
 Object::New(isolate, v8::Null(isolate), nullptr, nullptr, 0);
-for (int i = 0; i < raw_attributes->Length(); i += 3) {
+for (int i = 0; i < raw_attributes->Length(); i += elements_per_attribute) {
  attributes
  ->Set(realm->context(),
  raw_attributes->Get(realm->context(), i).As<String>(),
@@ -304,7 +308,7 @@ void ModuleWrap::Link(const FunctionCallbackInfo<Value>& args) {
 
  Local<FixedArray> raw_attributes = module_request->GetImportAssertions();
  Local<Object> attributes =
-createImportAttributesContainer(realm, isolate, raw_attributes);
+createImportAttributesContainer(realm, isolate, raw_attributes, 3);
 
  Local<Value> argv[] = {
  specifier,
@@ -588,7 +592,7 @@ static MaybeLocal<Promise> ImportModuleDynamically(
  }
 
  Local<Object> attributes =
-createImportAttributesContainer(realm, isolate, import_attributes);
+createImportAttributesContainer(realm, isolate, import_attributes, 2);
 
  Local<Value> import_args[] = {
  id,

test/es-module/test-esm-import-attributes-errors.js

@@ -26,6 +26,11 @@ async function test() {
 {code: 'ERR_IMPORT_ATTRIBUTE_TYPE_INCOMPATIBLE'}
 );
 
+awaitrejects(
+import(jsModuleDataUrl,{with: {type: 'json',other: 'unsupported'}}),
+{code: 'ERR_IMPORT_ATTRIBUTE_TYPE_INCOMPATIBLE'}
+);
+
 awaitrejects(
 import(jsModuleDataUrl,{with: {type: 'unsupported'}}),
 {code: 'ERR_IMPORT_ATTRIBUTE_UNSUPPORTED'}

test/es-module/test-esm-import-attributes-errors.mjs

@@ -21,6 +21,11 @@ await rejects(
 {code: 'ERR_IMPORT_ATTRIBUTE_TYPE_INCOMPATIBLE'}
 );
 
+awaitrejects(
+import(jsModuleDataUrl,{with: {type: 'json',other: 'unsupported'}}),
+{code: 'ERR_IMPORT_ATTRIBUTE_TYPE_INCOMPATIBLE'}
+);
+
 awaitrejects(
 import(import.meta.url,{with: {type: 'unsupported'}}),
 {code: 'ERR_IMPORT_ATTRIBUTE_UNSUPPORTED'}