fix: keep unreferenced arrays (#118)

j4k0xb · Sep 30, 2024 · 4ecd7e9 · 4ecd7e9
1 parent 10651d3
commit 4ecd7e9
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/packages/webcrack/src/deobfuscate/string-array.ts b/packages/webcrack/src/deobfuscate/string-array.ts
@@ -87,7 +87,8 @@ export function findStringArray(ast: t.Node): StringArray | undefined {
         m.fromCapture(arrayIdentifier),
         m.numericLiteral(m.matcher((value) => value < length)),
       );
-      if (!isReadonlyObject(binding, memberAccess)) return;
+      if (!binding.referenced || !isReadonlyObject(binding, memberAccess))
+        return;
 
       inlineArrayElements(arrayExpression.current!, binding.referencePaths);
       path.remove();

diff --git a/packages/webcrack/src/deobfuscate/test/samples/simple-string-array.js b/packages/webcrack/src/deobfuscate/test/samples/simple-string-array.js
@@ -8,4 +8,7 @@ console[arr2[0]](arr2[1]);
 
 const arr3 = ["requ", "m", "0x2649a392", "ire", "8", "v"];
 const vm = eval(arr3[0] + arr3[3] + "(\"" + arr3[5] + "" + arr3[1] + "\")");
-const v8 = eval(arr3[0] + arr3[3] + "(\"" + arr3[5] + "" + arr3[4] + "\")");
+const v8 = eval(arr3[0] + arr3[3] + "(\"" + arr3[5] + "" + arr3[4] + "\")");
+
+// ignore unreferenced array
+const arr4 = ['log', 'Hello, World!'];
diff --git a/packages/webcrack/src/deobfuscate/test/samples/simple-string-array.js.snap b/packages/webcrack/src/deobfuscate/test/samples/simple-string-array.js.snap
@@ -5,4 +5,7 @@ const arr2 = ["log", "Hello, World!"];
 arr2[0] = "warn";
 console[arr2[0]](arr2[1]);
 const vm = eval("require(\"vm\")");
-const v8 = eval("require(\"v8\")");
+const v8 = eval("require(\"v8\")");
+
+// ignore unreferenced array
+const arr4 = ["log", "Hello, World!"];