Class: Spektr::Checks::Deserialize

Inherits:
Base
  • Object
show all
Defined in:
lib/spektr/checks/deserialize.rb

Instance Attribute Summary

Attributes inherited from Base

#name

Instance Method Summary collapse

Methods inherited from Base

#app_version_between?, #dupe?, #full_receiver, #model_attribute?, #receivers_for, #should_run?, #target_affected?, #user_input?, #version_affected, #version_between?, #warn!

Constructor Details

#initialize(app, target) ⇒ Deserialize

Returns a new instance of Deserialize.



5
6
7
8
9
10
 
# File 'lib/spektr/checks/deserialize.rb', line 5

def initialize(app, target)
  super
  @name = "Unsafe object deserialization"
  @type = "Insecure Deserialization"
  @targets = ["Spektr::Targets::Base", "Spektr::Targets::Controller", "Spektr::Targets::Routes", "Spektr::Targets::View"]
end

Instance Method Details

#check_csv ⇒ Object 



20
21
22
 
# File 'lib/spektr/checks/deserialize.rb', line 20

def check_csv
  check_method(:load, :CSV)
end

#check_marshal ⇒ Object 



31
32
33
34
35
 
# File 'lib/spektr/checks/deserialize.rb', line 31

def check_marshal
  [:load, :restore].each do |method|
    check_method(method, :Marshal)
  end
end

#check_method(method, receiver) ⇒ Object 



48
49
50
51
52
53
54
55
 
# File 'lib/spektr/checks/deserialize.rb', line 48

def check_method(method, receiver)
  calls = @target.find_calls(method, receiver)
  calls.each do |call|
    if user_input?(call.arguments.arguments.first)
      warn! @target, self, call.location, "#{receiver}.#{method} is called with user supplied value"
    end
  end
end

#check_oj ⇒ Object 



37
38
39
40
41
42
43
44
45
46
 
# File 'lib/spektr/checks/deserialize.rb', line 37

def check_oj
  check_method(:object_load, :Oj)
  safe_default = false
  safe_default = true if @target.find_calls(:mimic_JSON, :Oj).any?
  call = @target.find_calls(:default_options=, :Oj).last
  safe_default = true if call && call.arguments.arguments.first.elements.find{|e| e.key.unescaped == "mode" }.value.unescaped != "object"
  unless safe_default
    check_method(:load, :Oj)
  end
end

#check_yaml ⇒ Object

TODO: handle safe yaml



25
26
27
28
29
 
# File 'lib/spektr/checks/deserialize.rb', line 25

def check_yaml
  [:load_documents, :load_stream, :parse_documents, :parse_stream].each do |method|
    check_method(method, :YAML)
  end
end

#run ⇒ Object 



12
13
14
15
16
17
18
 
# File 'lib/spektr/checks/deserialize.rb', line 12

def run
  return unless super
  check_csv
  check_yaml
  check_marshal
  check_oj
end