jsobfu 0.2.0 → 0.2.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    OWVkOTRmZDM5ZDZmYzkwOGFhNGM4MDk0ZjQ3NzljZWVkMWI4OWQwMg==
+    ZjMwNzdiYThlZjA5YzdlZDVhZmUwYWE5MThhNzY1MDBiOGQ2ZTAxMg==
   data.tar.gz: !binary |-
-    MGQzZGZkMTAxZDUxZmJjZjNhNzc0YzE0NDFkNDQ0MzI4ZjJiYTc4MQ==
+    ZTY0ZDAyNDllMmRiNDRjZDkyYTFjOWYyNzllZGM3ZGYxNTRkMGVlYQ==
 SHA512:
   metadata.gz: !binary |-
-    ZjA5NzRkNWZmMzQ0MDA0ZTkxZWRjMjYyYzc1NWEwZDE5YzcwMDNlZTZiN2E3
-    YTczZmQzZGMxODk1MzBlOTY5NjQwYzIxNTNmZTk5MjdlMzBlZmQ1ZGFiODRm
-    YThhYjAyMjhlYzhiZjliNzg2ZGZlOTc2ZmEwNDliZWNlOTg4MzU=
+    ZDA0Mjg0Y2YyODg1YTAxYmE2YzlmNDQ3NDllMzI0ZDE5YTgxZjdiYjM0OGU2
+    M2VkMzdhZDE3ZjI0MjA2OThkOGFkZWM4NGZhM2E3OTdmMWVmYzcxNDljMTYw
+    YTM0Zjg5NzJhZTU4MzA5NjA3OTc2NDdiNzY1MzgzMjRmYzJmNDk=
   data.tar.gz: !binary |-
-    NzhkYTM0OTRlM2EwYzgwNzQ2NmI2MGNhOTUzMDVmMjYzYzk4YmQyOTk3Y2Y2
-    ZmJiMTQyZDAwZGJhZGM2YzhlMzQzODAxMjA1MjlhYjE5NDA2NWEzODI0MWQ2
-    M2JmMzEzMjkyMzhjMWJkNWMwMmQyODU1Zjg4MzcyZWU1MTRjMzQ=
+    MjRkMmJhMDhjZDVjZjBjNjIzYWFlMjUwYmU0Y2VjYTgzZTdiODcwZDhmMDQ5
+    YjNjNTRlNWUwOWJiYmE4ZDVmZWMxNjVmZTg1ZTFmYjkwZDVjYThjNGViYTY2
+    YjhlOTgyOGQ2MDNjZTA5ZWIwOGJkZDljN2IxYTNlNjlmNGEwZGQ=

data/lib/jsobfu/obfuscator.rb

@@ -12,7 +12,10 @@ class JSObfu::Obfuscator < JSObfu::ECMANoWhitespaceVisitor
   attr_reader :global
 
   # unresolved lookups are rewritten as property lookups on the global object
-  DEFAULT_GLOBAL = 'window'
+  DEFAULT_GLOBAL  = 'window'
+
+  # some "global" functions are actually keywords, like void(5)
+  BUILTIN_METHODS = ['void']
 
   # @param opts [Hash] the options hash
   # @option opts [JSObfu::Scope] :scope the optional scope to save vars to
@@ -86,6 +89,10 @@ class JSObfu::Obfuscator < JSObfu::ECMANoWhitespaceVisitor
   # object (like "document"), and hence will not be obfuscated.
   #
   def visit_ResolveNode(o)
+    if is_builtin_method?(o.value)
+      return super
+    end
+
     new_val = rename_var(o.value, :generate => false)
 
     if new_val
@@ -151,4 +158,8 @@ class JSObfu::Obfuscator < JSObfu::ECMANoWhitespaceVisitor
     @renames[var_name] = scope.rename_var(var_name, opts)
   end
 
+  def is_builtin_method?(method)
+    BUILTIN_METHODS.include?(method)
+  end
+
 end

data/spec/jsobfu/obfuscator_spec.rb

@@ -31,4 +31,20 @@ describe JSObfu::Obfuscator do
     end
   end
 
+  describe 'when encountering the void() keyword' do
+
+    let(:opts) { { iterations: 3 } }
+    let(:simple_js) { "void(null);" }
+    let(:simple_ast) { RKelly::Parser.new.parse(simple_js) }
+
+    it 'does not rewrite the void() call as a global property lookup' do
+      expect(obfuscator.accept(simple_ast).to_s).not_to include("window")
+    end
+
+    it 'does not obfuscate the keyword void' do
+      expect(obfuscator.accept(simple_ast).to_s).to include("void")
+    end
+
+  end
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jsobfu
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - James Lee