web: safely handle immutable globals (aduros#779)

JerwuQu · Dec 3, 2024 · fc58c88 · fc58c88
1 parent 7e92c5e
commit fc58c88
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/runtimes/web/src/netplay/index.ts b/runtimes/web/src/netplay/index.ts
@@ -252,10 +252,10 @@ export class Netplay {
         const localPeerId = await this.peerMgr.localPeerId;
 
         const loc = window.location;
-        if (loc.protocol == "file:"
+        if (!DEV_NETPLAY && (loc.protocol == "file:"
             || loc.hostname == "localhost"
             || loc.hostname == "127.0.0.1"
-            || loc.hostname == "wasm4.org") {
+            || loc.hostname == "wasm4.org")) {
             return `https://wasm4.org/netplay/#${localPeerId}`;
         }
         const url = new URL(loc.href);

diff --git a/runtimes/web/src/state.ts b/runtimes/web/src/state.ts
@@ -38,7 +38,10 @@ export class State {
         for (const exName in runtime.wasm!.exports) {
             const exInst = runtime.wasm!.exports[exName]
             if (exInst instanceof WebAssembly.Global && exName in this.globals) {
-                exInst.value = this.globals[exName];
+                try {
+                    // this will fail for immutable globals that have been exported, but is safe
+                    exInst.value = this.globals[exName];
+                } catch {}
             }
         }