fix(Deobfuscator): return if Node is null in extractCounterIdentifiers (

#296)
NodeSecure · Aug 16, 2024 · 81e44c5 · 81e44c5
1 parent b69ced9
commit 81e44c5
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/src/Deobfuscator.js b/src/Deobfuscator.js
@@ -63,6 +63,9 @@ export class Deobfuscator {
    * @param {*} node
    */
   #extractCounterIdentifiers(nc, node) {
+    if (node === null) {
+      return;
+    }
     const { type } = nc;
 
     switch (type) {

diff --git a/test/issues/295-deobfuscator-function-declaration-id-null.spec.js b/test/issues/295-deobfuscator-function-declaration-id-null.spec.js
@@ -0,0 +1,16 @@
+// Import Node.js Dependencies
+import { test } from "node:test";
+
+// Import Internal Dependencies
+import { AstAnalyser } from "../../index.js";
+
+/**
+ * @see https://github.com/NodeSecure/js-x-ray/issues/295
+ */
+test("Deobfuscator.#extractCounterIdentifiers should not throw if FunctionDeclaration id is null", () => {
+  new AstAnalyser().analyse(`
+    export default async function (app) {
+      app.loaded = true
+    }
+  `);
+});