diff options
author | Jeremy Kemper <jeremy@bitsweat.net> | 2009-11-09 19:30:02 -0800 |
---|---|---|
committer | Jeremy Kemper <jeremy@bitsweat.net> | 2009-11-09 19:30:02 -0800 |
commit | 08f7c4dd8951053c443371f786be59d04448c225 (patch) | |
tree | 0365f3744e40e79e1421872ec1687070c88ede36 /activesupport/lib/active_support/core_ext/rexml.rb | |
parent | 703d31c20a7b531053d5c009972a89e857b07376 (diff) | |
download | rails-08f7c4dd8951053c443371f786be59d04448c225.tar.gz rails-08f7c4dd8951053c443371f786be59d04448c225.tar.bz2 rails-08f7c4dd8951053c443371f786be59d04448c225.zip |
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.
Diffstat (limited to 'activesupport/lib/active_support/core_ext/rexml.rb')
-rw-r--r-- | activesupport/lib/active_support/core_ext/rexml.rb | 43 |
1 files changed, 43 insertions, 0 deletions
diff --git a/activesupport/lib/active_support/core_ext/rexml.rb b/activesupport/lib/active_support/core_ext/rexml.rb new file mode 100644 index 0000000000..5288b639a6 --- /dev/null +++ b/activesupport/lib/active_support/core_ext/rexml.rb @@ -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 |