Class: Brakeman::CheckSimpleFormat

Inherits:
CheckCrossSiteScripting
  • Object
show all
Defined in:
lib/brakeman/checks/check_simple_format.rb

Instance Method Summary collapse

Constructor Details

#initialize(*args) ⇒ CheckSimpleFormat

Returns a new instance of CheckSimpleFormat.



8
9
10
11
 
# File 'lib/brakeman/checks/check_simple_format.rb', line 8

def initialize *args
  super
  @found_any = false
end

Instance Method Details

#check_simple_format_usage ⇒ Object 



35
36
37
38
39
40
41
42
43
 
# File 'lib/brakeman/checks/check_simple_format.rb', line 35

def check_simple_format_usage
  tracker.find_call(:target => false, :method => :simple_format).each do |result|
    @matched = false
    process_call result[:call]
    if @matched
      warn_on_simple_format result, @matched
    end
  end
end

#generic_warning ⇒ Object 



23
24
25
26
27
28
29
30
31
32
33
 
# File 'lib/brakeman/checks/check_simple_format.rb', line 23

def generic_warning
  message = msg(msg_version(rails_version), " has a vulnerability in ", msg_code("simple_format"), " ", msg_cve("CVE-2013-6416"), ". Upgrade to ", msg_version("4.0.2"))

  warn :warning_type => "Cross-Site Scripting",
    :warning_code => :CVE_2013_6416,
    :message => message,
    :confidence => :medium,
    :gem_info => gemfile_or_environment,
    :link_path => "https://groups.google.com/d/msg/ruby-security-ann/5ZI1-H5OoIM/ZNq4FoR2GnIJ",
    :cwe_id => [79]
end

#process_call(exp) ⇒ Object 



45
46
47
48
49
 
# File 'lib/brakeman/checks/check_simple_format.rb', line 45

def process_call exp
  @mark = true
  actually_process_call exp
  exp
end

#run_check ⇒ Object 



13
14
15
16
17
18
19
20
21
 
# File 'lib/brakeman/checks/check_simple_format.rb', line 13

def run_check
  if version_between? "4.0.0", "4.0.1"
    @inspect_arguments = true
    @ignore_methods = Set[:h, :escapeHTML]

    check_simple_format_usage
    generic_warning unless @found_any
  end
end

#warn_on_simple_format(result, match) ⇒ Object 



51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
# File 'lib/brakeman/checks/check_simple_format.rb', line 51

def warn_on_simple_format result, match
  return unless original? result

  @found_any = true

  warn :result => result,
    :warning_type => "Cross-Site Scripting",
    :warning_code => :CVE_2013_6416_call,
    :message => msg("Values passed to ", msg_code("simple_format"), " are not safe in ", msg_version(rails_version)),
    :confidence => :high,
    :link_path => "https://groups.google.com/d/msg/ruby-security-ann/5ZI1-H5OoIM/ZNq4FoR2GnIJ",
    :user_input => match,
    :cwe_id => [79]
end