From 5ce41a960a44aff20d3d3c5586b3ca847bc1b839 Mon Sep 17 00:00:00 2001
From: Justin Collins <jcollins@attinteractive.com>
Date: Wed, 30 Nov 2011 16:17:33 -0800
Subject: [PATCH] Don't dup nils

---
 lib/brakeman/processors/alias_processor.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/brakeman/processors/alias_processor.rb b/lib/brakeman/processors/alias_processor.rb
index 6afacfc1..fb27718b 100644
--- a/lib/brakeman/processors/alias_processor.rb
+++ b/lib/brakeman/processors/alias_processor.rb
@@ -364,7 +364,8 @@ class Brakeman::AliasProcessor < SexpProcessor
   def only_ivars
     res = SexpProcessor::Environment.new
     env.all.each do |k, v|
-      res[k] = v.dup if k.node_type == :ivar
+      #TODO Why would this have nil values?
+      res[k] = v.dup if k.node_type == :ivar and not v.nil?
     end
     res
   end
-- 
GitLab