Revert "the REXML security fix is not needed for Ruby >= 1.8.7"
Still required on older 1.8.7 patchlevels. This reverts commit a48f49e56bf53e0a2386da898576ef12d5258358.
This commit is contained in:
parent
703d31c20a
commit
08f7c4dd89
43
activesupport/lib/active_support/core_ext/rexml.rb
Normal file
43
activesupport/lib/active_support/core_ext/rexml.rb
Normal file
@ -0,0 +1,43 @@
|
||||
require 'active_support/core_ext/kernel/reporting'
|
||||
|
||||
# Fixes the rexml vulnerability disclosed at:
|
||||
# http://www.ruby-lang.org/en/news/2008/08/23/dos-vulnerability-in-rexml/
|
||||
# This fix is identical to rexml-expansion-fix version 1.0.1
|
||||
require 'rexml/rexml'
|
||||
|
||||
# Earlier versions of rexml defined REXML::Version, newer ones REXML::VERSION
|
||||
unless (defined?(REXML::VERSION) ? REXML::VERSION : REXML::Version) > "3.1.7.2"
|
||||
silence_warnings { require 'rexml/document' }
|
||||
|
||||
# REXML in 1.8.7 has the patch but early patchlevels didn't update Version from 3.1.7.2.
|
||||
unless REXML::Document.respond_to?(:entity_expansion_limit=)
|
||||
silence_warnings { require 'rexml/entity' }
|
||||
|
||||
module REXML #:nodoc:
|
||||
class Entity < Child #:nodoc:
|
||||
undef_method :unnormalized
|
||||
def unnormalized
|
||||
document.record_entity_expansion! if document
|
||||
v = value()
|
||||
return nil if v.nil?
|
||||
@unnormalized = Text::unnormalize(v, parent)
|
||||
@unnormalized
|
||||
end
|
||||
end
|
||||
class Document < Element #:nodoc:
|
||||
@@entity_expansion_limit = 10_000
|
||||
def self.entity_expansion_limit= val
|
||||
@@entity_expansion_limit = val
|
||||
end
|
||||
|
||||
def record_entity_expansion!
|
||||
@number_of_expansions ||= 0
|
||||
@number_of_expansions += 1
|
||||
if @number_of_expansions > @@entity_expansion_limit
|
||||
raise "Number of entity expansions exceeded, processing aborted."
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
@ -5,6 +5,7 @@
|
||||
# DateTime to_date, to_datetime, xmlschema
|
||||
# Enumerable group_by, each_with_object, none?
|
||||
# Process Process.daemon
|
||||
# REXML security fix
|
||||
# String ord
|
||||
# Time to_date, to_time, to_datetime
|
||||
require 'active_support'
|
||||
@ -13,4 +14,5 @@
|
||||
require 'active_support/core_ext/enumerable'
|
||||
require 'active_support/core_ext/process/daemon'
|
||||
require 'active_support/core_ext/string/conversions'
|
||||
require 'active_support/core_ext/rexml'
|
||||
require 'active_support/core_ext/time/conversions'
|
||||
|
Loading…
Reference in New Issue
Block a user