summaryrefslogtreecommitdiffstats
path: root/Doc
diff options
context:
space:
mode:
authorAntoine Pitrou <solipsis@pitrou.net>2011-12-05 00:22:03 (GMT)
committerAntoine Pitrou <solipsis@pitrou.net>2011-12-05 00:22:03 (GMT)
commit9b1ec97867ffd46eb134ea63012a0d9a57ac0b5b (patch)
tree32b00019d9becca0222e3bbfe75b7eda9fd049b1 /Doc
parentb53e2c4900d33b4d273358a95d993954d35ee52f (diff)
parent534e253cb6ca13fa790b9fbe5359c88e779722f6 (diff)
downloadcpython-9b1ec97867ffd46eb134ea63012a0d9a57ac0b5b.zip
cpython-9b1ec97867ffd46eb134ea63012a0d9a57ac0b5b.tar.gz
cpython-9b1ec97867ffd46eb134ea63012a0d9a57ac0b5b.tar.bz2
Remove reference to the base64 encoding.
Diffstat (limited to 'Doc')
-rw-r--r--Doc/howto/unicode.rst29
1 files changed, 6 insertions, 23 deletions
diff --git a/Doc/howto/unicode.rst b/Doc/howto/unicode.rst
index 77fcd26..045fd33 100644
--- a/Doc/howto/unicode.rst
+++ b/Doc/howto/unicode.rst
@@ -552,7 +552,6 @@ should only be used on systems where undecodable file names can be present,
i.e. Unix systems.
-
Tips for Writing Unicode-aware Programs
---------------------------------------
@@ -572,28 +571,12 @@ you do e.g. ``str + bytes``, a :exc:`TypeError` is raised for this expression.
When using data coming from a web browser or some other untrusted source, a
common technique is to check for illegal characters in a string before using the
string in a generated command line or storing it in a database. If you're doing
-this, be careful to check the string once it's in the form that will be used or
-stored; it's possible for encodings to be used to disguise characters. This is
-especially true if the input data also specifies the encoding; many encodings
-leave the commonly checked-for characters alone, but Python includes some
-encodings such as ``'base64'`` that modify every single character.
-
-For example, let's say you have a content management system that takes a Unicode
-filename, and you want to disallow paths with a '/' character. You might write
-this code::
-
- def read_file(filename, encoding):
- if '/' in filename:
- raise ValueError("'/' not allowed in filenames")
- unicode_name = filename.decode(encoding)
- with open(unicode_name, 'r') as f:
- # ... return contents of file ...
-
-However, if an attacker could specify the ``'base64'`` encoding, they could pass
-``'L2V0Yy9wYXNzd2Q='``, which is the base-64 encoded form of the string
-``'/etc/passwd'``, to read a system file. The above code looks for ``'/'``
-characters in the encoded form and misses the dangerous character in the
-resulting decoded form.
+this, be careful to check the decoded string, not the encoded bytes data;
+some encodings may have interesting properties, such as not being bijective
+or not being fully ASCII-compatible. This is especially true if the input
+data also specifies the encoding, since the attacker can then choose a
+clever way to hide malicious text in the encoded bytestream.
+
References
----------