diff options
| -rw-r--r-- | ChangeLog | 5 | ||||
| -rw-r--r-- | doc/security.rdoc | 20 |
2 files changed, 5 insertions, 20 deletions
@@ -1,3 +1,8 @@ +Wed Feb 6 03:45:19 2013 Zachary Scott <zachary@zacharyscott.net> + + * doc/security.rdoc: Remove documentation for unsafe CSV.load which + was deleted in r39077 + Wed Feb 6 03:27:19 2013 James Edward Gray II <james@graysoftinc.com> * lib/csv.rb: Remove the dangerous serialization feature. diff --git a/doc/security.rdoc b/doc/security.rdoc index c98011313d..566920a5c1 100644 --- a/doc/security.rdoc +++ b/doc/security.rdoc @@ -54,26 +54,6 @@ deserialized: Because of this, many of the security considerations applying to Marshal are also applicable to YAML. Do not use YAML to deserialize untrusted data. -== CSV - -Never use +CSV.load+ to parse untrusted CSV data. +CSV.load+ shares many of the -same issues as YAML and Marshal in that it will deserialize to arbitrary -classes: - - class,ERB - @src - puts `uname` - -However, CSV's +load+ method is significantly more dangerous than Marshal and -YAML as it will call arbitrary methods with attacker controlled arguments in -some cases: - - class,Object - eval - puts `uname` - -If you need to parse user supplied CSV data, use +CSV.parse+ instead. - == Symbols Symbols are often seen as syntax sugar for simple strings, but they play a much |