diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2011-10-03 14:23:10 -0700 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2011-10-03 14:23:10 -0700 |
commit | 2422a9fc3aeff3c60c6510efbf655a34218c7605 (patch) | |
tree | 525da292089cf01cb3aabdefa7047e87c76a4712 /lib | |
parent | 68a91d5e6446cacac9ded1c03381244557b02e2e (diff) | |
download | psych-2422a9fc3aeff3c60c6510efbf655a34218c7605.zip |
Avoid accepting incorrect floating point values.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/psych/scalar_scanner.rb | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/lib/psych/scalar_scanner.rb b/lib/psych/scalar_scanner.rb index b92d3c0..3e8acbb 100644 --- a/lib/psych/scalar_scanner.rb +++ b/lib/psych/scalar_scanner.rb @@ -7,6 +7,12 @@ module Psych # Taken from http://yaml.org/type/timestamp.html TIME = /^\d{4}-\d{1,2}-\d{1,2}([Tt]|\s+)\d{1,2}:\d\d:\d\d(\.\d*)?(\s*Z|[-+]\d{1,2}(:\d\d)?)?/ + # Taken from http://yaml.org/type/float.html + FLOAT = /^(?:[-+]?([0-9][0-9_,]*)?\.[0-9.]*([eE][-+][0-9]+)?(?# base 10) + |[-+]?[0-9][0-9_,]*(:[0-5]?[0-9])+\.[0-9_]*(?# base 60) + |[-+]?\.(inf|Inf|INF)(?# infinity) + |\.(nan|NaN|NAN)(?# not a number))$/x + # Create a new scanner def initialize @string_cache = {} @@ -67,10 +73,14 @@ module Psych i += (n.to_f * 60 ** (e - 2).abs) end i + when FLOAT + return Float(string.gsub(/[,_]/, '')) rescue ArgumentError + + @string_cache[string] = true + string else if string.count('.') < 2 return Integer(string.gsub(/[,_]/, '')) rescue ArgumentError - return Float(string.gsub(/[,_]/, '')) rescue ArgumentError end @string_cache[string] = true |