watobo 0.9.19 → 0.9.20

data/modules/active/sqlinjection/sql_boolean.rb

@@ -1,24 +1,12 @@
-# .
+#.
 # sql_boolean.rb
-# 
-# Copyright 2013 by siberas, http://www.siberas.de
-# 
-# This file is part of WATOBO (Web Application Tool Box)
-#        http://watobo.sourceforge.com
-# 
-# WATOBO is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation version 2 of the License.
-# 
-# WATOBO is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-# 
-# You should have received a copy of the GNU General Public License
-# along with WATOBO; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
-# .
+#.
+# Copyright 2014 by siberas, http://www.siberas.de
+# This file is part of WATOBO (Web Application Tool Box) http://watobo.sourceforge.com
+# WATOBO is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation version 2 of the License.
+# WATOBO is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# You should have received a copy of the GNU General Public License along with WATOBO; if not, write to the Free Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
 require 'digest/md5'
 require 'digest/sha1'
 
@@ -97,7 +85,7 @@ EOF
             #  Check GET-Parameters
             #
             begin
-              chat.request.parameters(:url, :wwwform, :xml) do |param|
+              chat.request.parameters(:url, :wwwform, :xml, :json) do |param|
                 checks = []
                 @prefs.each do |p|
                   @fins.each do |f|

data/modules/active/sqlinjection/sqli_error.rb

@@ -1,24 +1,12 @@
-# .
+#.
 # sqli_error.rb
-# 
-# Copyright 2013 by siberas, http://www.siberas.de
-# 
-# This file is part of WATOBO (Web Application Tool Box)
-#        http://watobo.sourceforge.com
-# 
-# WATOBO is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation version 2 of the License.
-# 
-# WATOBO is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-# 
-# You should have received a copy of the GNU General Public License
-# along with WATOBO; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
-# .
+#.
+# Copyright 2014 by siberas, http://www.siberas.de
+# This file is part of WATOBO (Web Application Tool Box) http://watobo.sourceforge.com
+# WATOBO is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation version 2 of the License.
+# WATOBO is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# You should have received a copy of the GNU General Public License along with WATOBO; if not, write to the Free Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
 # @private 
 module Watobo#:nodoc: all
   module Modules
@@ -89,7 +77,7 @@ EOF
           def generateChecks(chat)
             
             begin
-              chat.request.parameters( :url, :wwwform ) do |parm|
+              chat.request.parameters( :url, :wwwform, :json ) do |parm|
                 # puts "#{Module.nesting[0].name}: run check on chat-id (#{chat.id}) with parm (#{parm})"
                 #@sql_checks.each do |check, pattern|
                 test_values = []
@@ -109,7 +97,7 @@ EOF
                     parm.value = check
                     test.set parm
                     
-                    puts test
+                    #puts test
                     # fire it up!
                     #puts req_copy
                     test_request,test_response = doRequest(test)

data/modules/active/sqlinjection/sqli_timing.rb

@@ -1,243 +1,231 @@
-# .
+#.
 # sqli_timing.rb
-# 
-# Copyright 2013 by siberas, http://www.siberas.de
-# 
-# This file is part of WATOBO (Web Application Tool Box)
-#        http://watobo.sourceforge.com
-# 
-# WATOBO is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation version 2 of the License.
-# 
-# WATOBO is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-# 
-# You should have received a copy of the GNU General Public License
-# along with WATOBO; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
-# .
-require 'digest/md5'
-require 'digest/sha1'
+#.
+# Copyright 2014 by siberas, http://www.siberas.de
+# This file is part of WATOBO (Web Application Tool Box) http://watobo.sourceforge.com
+# WATOBO is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation version 2 of the License.
+# WATOBO is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# You should have received a copy of the GNU General Public License along with WATOBO; if not, write to the Free Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 
+require 'digest/md5'
+require 'digest/sha1'
+
 # @private 
-module Watobo#:nodoc: all
-  module Modules
-    module Active
-      module Sqlinjection
-        
-        
-        class Sqli_timing < Watobo::ActiveCheck
-          @info.update(
-                         :check_name => 'Time-based SQL Injection',    # name of check which briefly describes functionality, will be used for tree and progress views
-            :check_group => AC_GROUP_SQL,
-            :description => "Checking each parameter for SQL-Injection flaws using timing techniques.",   # description of checkfunction
-            :author => "Andreas Schmidt", # author of check
-            :version => "0.9"   # check version
-            )
-            
-            threat =<<'EOF'
-SQL Injection is an attack technique used to exploit applications that construct SQL statements from user-supplied input. 
-When successful, the attacker is able to change the logic of SQL statements executed against the database.
-Structured Query Language (SQL) is a specialized programming language for sending queries to databases. 
-The SQL programming language is both an ANSI and an ISO standard, though many database products supporting SQL do so with 
-proprietary extensions to the standard language. Applications often use user-supplied data to create SQL statements. 
-If an application fails to properly construct SQL statements it is possible for an attacker to alter the statement structure 
-and execute unplanned and potentially hostile commands. When such commands are executed, they do so under the context of the user 
-specified by the application executing the statement. This capability allows attackers to gain control of all database resources 
-accessible by that user, up to and including the ability to execute commands on the hosting system.
-
-Source: http://projects.webappsec.org/SQL-Injection
-EOF
-            
-            measure = "All user input must be escaped and/or filtered thoroughly before the sql statement is put together. Additionally prepared statements should be used."
-            
-            @finding.update(
-                            :threat => threat,        # thread of vulnerability, e.g. loss of information
-                            :class => "SQL-Injection",    # vulnerability class, e.g. Stored XSS, SQL-Injection, ...
-                            :type => FINDING_TYPE_VULN,         # FINDING_TYPE_HINT, FINDING_TYPE_INFO, FINDING_TYPE_VULN
-                            :rating => VULN_RATING_CRITICAL,
-                            :measure => measure
-                            )
-            
-            
-          def initialize(project, prefs={})
-            super(project, prefs)
-            
-          end
-          
-          def generateChecks(chat)
-            sql_timing_commands = [
-              'and sleep(SLEEP_TIME)',
-              'and 1 in (select BENCHMARK(20000000,MD5(CHAR(97))))',
-              'and waitfor delay \'0:0:SLEEP_TIME\''
-            ]
-            
-            sqli_vectors = [ 
-              '', 
-              '\'', 
-              '\'))', 
-              '\')))', 
-              ')', 
-              '))', 
-              ')))'
-              ]
-              
-              sql_terminators = [
-                '', '--', ';--'
-              ]
-              
-              sqli_patterns = []
-              sqli_vectors.each do |v|
-                 sql_timing_commands.each do |stc|
-                   sql_terminators.each do |sts|
-                     sqli_patterns << "#{v} #{stc}#{sts}"
-                   end
-                 end  
-              end
-              
-              check_parms = []
-              
-              urlParmNames(chat).each do |parm|
-                pval = chat.request.get_parm_value(parm)
-                check_parms << { :name => parm, :value => pval, :type => :url }
-              end
-              
-              postParmNames(chat).each do |parm|
-                pval = chat.request.post_parm_value(parm)
-                check_parms << { :name => parm, :value => pval, :type => :form }
-              end
-              
-             
-                checker = proc {
-                   test_request = nil
-                   test_response = nil
-                   output = ""
-                   
-                   check_parms.each do |parm|
-                 # first get multiple response times
-                      rtimes = []
-                     
-                      timing_response = nil
-                      
-                      vulnerable = false
-                      
-                      4.times do
-                        test = chat.copyRequest
-                         start = Time.now().to_i
-                         timing_request, timing_response = doRequest(test,:default => true)
-                         stop = Time.now().to_i
-                         rtimes << ( stop - start )
-                      
-                      end
-                      # now calculate the average time
-                      average_t = rtimes.inject(:+) / rtimes.length
-                      max_t = rtimes.max > 5 ? rtimes.max : 5
-                     # puts "Analyzing timing behaviour ..."
-                     # rtimes.each do |t|
-                     #   puts t.to_s
-                     # end
-                     # puts "Average Response Time: #{average_t}s (max #{max_t}s)"                     
-                      
-                 #     time_to_sleep = 4 * max_t
-                      time_to_sleep = max_t
-                      #timeout_t = time_to_sleep + average_t
-                      timeout_t = 2 * time_to_sleep
-                       
-                      test_value = ""
-                      test = nil
-                      log_request = nil
-                      max_timeouts = 2
-                      timeout_counter = 0
-                      sqli_start = sqli_stop = 0
-                      
-                      sqli_patterns.each do |sql|
-                        timeout_counter = 0
-                        output = ""
-                        break if vulnerable
-                        begin
-                           sqli_start = Time.now().to_i
-                           timeout(timeout_t) do
-                             test = chat.copyRequest
-                             # also need to check if altered parm will change response
-                             test_value = CGI.escape("#{parm[:value]}#{sql.gsub(/SLEEP_TIME/, time_to_sleep.to_s)}")
-                             case parm[:type]
-                               when :url     
-                                 test.replace_get_parm(parm[:name], test_value)
-                               when :form
-                                 test.replace_post_parm(parm[:name], test_value)
-                             end
-                             
-                             test_request, test_response = doRequest(test,:default => true)
-                             sqli_stop = Time.now().to_i
-                           end
-                         rescue Timeout::Error
-                           timeout_counter += 1
-                      #     puts "[#{self}] Hit Timeout after #{timeout_t} seconds (#{timeout_counter})."
-                      #     puts test
-                      #     puts 
-                      #     puts "... retry after #{max_t} seconds ..."
-                           sleep max_t
-                           retry unless timeout_counter > 2
-                           sqli_stop = Time.now().to_i
-                           output << "Hit Timeout after #{sqli_start - sqli_stop} seconds\n"
-                      #     puts "* redo request with sleep_time=0 to get an apropriate server response ..."
-                           test = chat.copyRequest
-                           test_value = CGI.escape("#{parm[:value]}#{sql.gsub(/SLEEP_TIME/, "0")}")     
-                             case parm[:type]
-                               when :url     
-                             test.replace_get_parm(parm[:name], test_value)
-                             when :form
-                               test.replace_post_parm(parm[:name], test_value)
-                             end
-                             
-                             dummy_request, test_response = doRequest(test, :default => true)
-                            
-                         rescue => bang
-                           puts bang
-                           puts bang.backtrace
-                         end    
-                         
-                         
-                         duration = sqli_stop - sqli_start
-                       #  puts duration
-                         if ( duration >= time_to_sleep )
-                           puts "Found time-based SQLi in parameter #{parm} !!!"
-                           puts "after #{duration}s / time-to-sleep #{time_to_sleep}s)"
-                           test_request.extend Watobo::Mixin::Parser::Url unless test_request.respond_to? :path
-                           path = "/" + test_request.path
-                           
-                           vulnerable = true
-                           output << "SleepTime: #{time_to_sleep}\nQuery Duration: #{duration}s" 
-                                                   
-                           addFinding( test_request, test_response,
-                                     :check_pattern => "#{test_value}",
-                                     :chat => chat,
-                                     :title => "[#{parm[:name]}] - #{path}",
-                                     :proof_pattern => "",
-                                     :test_item => parm[:name],
-                                     :class => "SQL-Injection (Time-based)",
-                                     :output => output               
-                                     )
-                            #readlines
-                          break 
-                         end
-                      end
-                        
-                        end
-                 
-                    [ test_request, test_response ]
-                  }
-                  yield checker
-                  
-                
-             end
-          end
-
-        # --> eo namespace    
-      end
-    end
-  end
-end
+module Watobo#:nodoc: all
+  module Modules
+    module Active
+      module Sqlinjection
+        
+        
+        class Sqli_timing < Watobo::ActiveCheck
+          @info.update(
+                         :check_name => 'Time-based SQL Injection',    # name of check which briefly describes functionality, will be used for tree and progress views
+            :check_group => AC_GROUP_SQL,
+            :description => "Checking each parameter for SQL-Injection flaws using timing techniques.",   # description of checkfunction
+            :author => "Andreas Schmidt", # author of check
+            :version => "0.9"   # check version
+            )
+            
+            threat =<<'EOF'
+SQL Injection is an attack technique used to exploit applications that construct SQL statements from user-supplied input. 
+When successful, the attacker is able to change the logic of SQL statements executed against the database.
+Structured Query Language (SQL) is a specialized programming language for sending queries to databases. 
+The SQL programming language is both an ANSI and an ISO standard, though many database products supporting SQL do so with 
+proprietary extensions to the standard language. Applications often use user-supplied data to create SQL statements. 
+If an application fails to properly construct SQL statements it is possible for an attacker to alter the statement structure 
+and execute unplanned and potentially hostile commands. When such commands are executed, they do so under the context of the user 
+specified by the application executing the statement. This capability allows attackers to gain control of all database resources 
+accessible by that user, up to and including the ability to execute commands on the hosting system.
+
+Source: http://projects.webappsec.org/SQL-Injection
+EOF
+            
+            measure = "All user input must be escaped and/or filtered thoroughly before the sql statement is put together. Additionally prepared statements should be used."
+            
+            @finding.update(
+                            :threat => threat,        # thread of vulnerability, e.g. loss of information
+                            :class => "SQL-Injection",    # vulnerability class, e.g. Stored XSS, SQL-Injection, ...
+                            :type => FINDING_TYPE_VULN,         # FINDING_TYPE_HINT, FINDING_TYPE_INFO, FINDING_TYPE_VULN
+                            :rating => VULN_RATING_CRITICAL,
+                            :measure => measure
+                            )
+            
+            
+          def initialize(project, prefs={})
+            super(project, prefs)
+            
+          end
+          
+          def generateChecks(chat)
+            sql_timing_commands = [
+              'and sleep(SLEEP_TIME)',
+              'and 1 in (select BENCHMARK(20000000,MD5(CHAR(97))))',
+              'and waitfor delay \'0:0:SLEEP_TIME\''
+            ]
+            
+            sqli_vectors = [ 
+              '', 
+              '\'', 
+              '\'))', 
+              '\')))', 
+              ')', 
+              '))', 
+              ')))'
+              ]
+              
+              sql_terminators = [
+                '', '--', ';--'
+              ]
+              
+              sqli_patterns = []
+              sqli_vectors.each do |v|
+                 sql_timing_commands.each do |stc|
+                   sql_terminators.each do |sts|
+                     sqli_patterns << "#{v} #{stc}#{sts}"
+                   end
+                 end  
+              end
+              
+              check_parms = []
+              
+              urlParmNames(chat).each do |parm|
+                pval = chat.request.get_parm_value(parm)
+                check_parms << { :name => parm, :value => pval, :type => :url }
+              end
+              
+              postParmNames(chat).each do |parm|
+                pval = chat.request.post_parm_value(parm)
+                check_parms << { :name => parm, :value => pval, :type => :form }
+              end
+              
+             
+                checker = proc {
+                   test_request = nil
+                   test_response = nil
+                   output = ""
+                   
+                   check_parms.each do |parm|
+                 # first get multiple response times
+                      rtimes = []
+                     
+                      timing_response = nil
+                      
+                      vulnerable = false
+                      
+                      4.times do
+                        test = chat.copyRequest
+                         start = Time.now().to_i
+                         timing_request, timing_response = doRequest(test,:default => true)
+                         stop = Time.now().to_i
+                         rtimes << ( stop - start )
+                      
+                      end
+                      # now calculate the average time
+                      average_t = rtimes.inject(:+) / rtimes.length
+                      max_t = rtimes.max > 5 ? rtimes.max : 5
+                     # puts "Analyzing timing behaviour ..."
+                     # rtimes.each do |t|
+                     #   puts t.to_s
+                     # end
+                     # puts "Average Response Time: #{average_t}s (max #{max_t}s)"                     
+                      
+                 #     time_to_sleep = 4 * max_t
+                      time_to_sleep = max_t
+                      #timeout_t = time_to_sleep + average_t
+                      timeout_t = 2 * time_to_sleep
+                       
+                      test_value = ""
+                      test = nil
+                      log_request = nil
+                      max_timeouts = 2
+                      timeout_counter = 0
+                      sqli_start = sqli_stop = 0
+                      
+                      sqli_patterns.each do |sql|
+                        timeout_counter = 0
+                        output = ""
+                        break if vulnerable
+                        begin
+                           sqli_start = Time.now().to_i
+                           timeout(timeout_t) do
+                             test = chat.copyRequest
+                             # also need to check if altered parm will change response
+                             test_value = CGI.escape("#{parm[:value]}#{sql.gsub(/SLEEP_TIME/, time_to_sleep.to_s)}")
+                             case parm[:type]
+                               when :url     
+                                 test.replace_get_parm(parm[:name], test_value)
+                               when :form
+                                 test.replace_post_parm(parm[:name], test_value)
+                             end
+                             
+                             test_request, test_response = doRequest(test,:default => true)
+                             sqli_stop = Time.now().to_i
+                           end
+                         rescue Timeout::Error
+                           timeout_counter += 1
+                      #     puts "[#{self}] Hit Timeout after #{timeout_t} seconds (#{timeout_counter})."
+                      #     puts test
+                      #     puts 
+                      #     puts "... retry after #{max_t} seconds ..."
+                           sleep max_t
+                           retry unless timeout_counter > 2
+                           sqli_stop = Time.now().to_i
+                           output << "Hit Timeout after #{sqli_start - sqli_stop} seconds\n"
+                      #     puts "* redo request with sleep_time=0 to get an apropriate server response ..."
+                           test = chat.copyRequest
+                           test_value = CGI.escape("#{parm[:value]}#{sql.gsub(/SLEEP_TIME/, "0")}")     
+                             case parm[:type]
+                               when :url     
+                             test.replace_get_parm(parm[:name], test_value)
+                             when :form
+                               test.replace_post_parm(parm[:name], test_value)
+                             end
+                             
+                             dummy_request, test_response = doRequest(test, :default => true)
+                            
+                         rescue => bang
+                           puts bang
+                           puts bang.backtrace
+                         end    
+                         
+                         
+                         duration = sqli_stop - sqli_start
+                       #  puts duration
+                         if ( duration >= time_to_sleep )
+                           puts "Found time-based SQLi in parameter #{parm} !!!"
+                           puts "after #{duration}s / time-to-sleep #{time_to_sleep}s)"
+                           test_request.extend Watobo::Mixin::Parser::Url unless test_request.respond_to? :path
+                           path = "/" + test_request.path
+                           
+                           vulnerable = true
+                           output << "SleepTime: #{time_to_sleep}\nQuery Duration: #{duration}s" 
+                                                   
+                           addFinding( test_request, test_response,
+                                     :check_pattern => "#{test_value}",
+                                     :chat => chat,
+                                     :title => "[#{parm[:name]}] - #{path}",
+                                     :proof_pattern => "",
+                                     :test_item => parm[:name],
+                                     :class => "SQL-Injection (Time-based)",
+                                     :output => output               
+                                     )
+                            #readlines
+                          break 
+                         end
+                      end
+                        
+                        end
+                 
+                    [ test_request, test_response ]
+                  }
+                  yield checker
+                  
+                
+             end
+          end
+
+        # --> eo namespace    
+      end
+    end
+  end
+end