Don't dup nils

5ce41a96 · Justin Collins · d78ce4ed · 5ce41a96
隐藏空白更改
内联并排

Showing with 2 addition and 1 deletion

lib/brakeman/processors/alias_processor.rb lib/brakeman/processors/alias_processor.rb +2 -1

未找到文件。
--- 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