webassembly/objjsproxy: Fix proxying in arguments to JS new function.

dpgeorge · dpgeorge · commit e860e32e24e1 · 2024-05-06T14:47:05.000+10:00
Signed-off-by: Damien George &lt;damien@micropython.org&gt;
diff --git a/ports/webassembly/objjsproxy.c b/ports/webassembly/objjsproxy.c
@@ -147,7 +147,7 @@ EM_JS(void, js_reflect_construct, (int f_ref, uint32_t n_args, uint32_t * args,
     const f = proxy_js_ref[f_ref];
     const as = [];
     for (let i = 0; i < n_args; ++i) {
-        as.push(proxy_convert_mp_to_js_obj_jsside(args + i * 4));
+        as.push(proxy_convert_mp_to_js_obj_jsside(args + i * 3 * 4));
     }
     const ret = Reflect.construct(f, as);
     proxy_convert_js_to_mp_obj_jsside(ret, out);
@@ -242,7 +242,7 @@ static mp_obj_t jsproxy_reflect_construct(size_t n_args, const mp_obj_t *args) {
     for (unsigned int i = 0; i < n_args; ++i) {
         proxy_convert_mp_to_js_obj_cside(args[i], &args_conv[i * PVN]);
     }
-    uint32_t out[3];
+    uint32_t out[PVN];
     js_reflect_construct(arg0, n_args, args_conv, out);
     return proxy_convert_js_to_mp_obj_cside(out);
 }

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ EM_JS(void, js_reflect_construct, (int f_ref, uint32_t n_args, uint32_t * args,`
`147`	`147`	`const f = proxy_js_ref[f_ref];`
`148`	`148`	`const as = [];`
`149`	`149`	`for (let i = 0; i < n_args; ++i) {`
`150`		`- as.push(proxy_convert_mp_to_js_obj_jsside(args + i * 4));`
	`150`	`+ as.push(proxy_convert_mp_to_js_obj_jsside(args + i * 3 * 4));`
`151`	`151`	`}`
`152`	`152`	`const ret = Reflect.construct(f, as);`
`153`	`153`	`proxy_convert_js_to_mp_obj_jsside(ret, out);`
`@@ -242,7 +242,7 @@ static mp_obj_t jsproxy_reflect_construct(size_t n_args, const mp_obj_t *args) {`
`242`	`242`	`for (unsigned int i = 0; i < n_args; ++i) {`
`243`	`243`	`proxy_convert_mp_to_js_obj_cside(args[i], &args_conv[i * PVN]);`
`244`	`244`	`}`
`245`		`- uint32_t out[3];`
	`245`	`+ uint32_t out[PVN];`
`246`	`246`	`js_reflect_construct(arg0, n_args, args_conv, out);`
`247`	`247`	`return proxy_convert_js_to_mp_obj_cside(out);`
`248`	`248`	`}`