diff options
author | mmenke <mmenke@chromium.org> | 2016-03-23 16:06:49 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-23 23:08:06 +0000 |
commit | 0a2a0e15f6741225fbce75cfc4fa07db0a00190d (patch) | |
tree | 505763ff952d83497ee88dfd5c2373c2b67e49bc /ash/display/display_color_manager_chromeos.cc | |
parent | a94e5359072a8a04163cfa6e0d061b27b1c5adc9 (diff) | |
download | chromium_src-0a2a0e15f6741225fbce75cfc4fa07db0a00190d.zip chromium_src-0a2a0e15f6741225fbce75cfc4fa07db0a00190d.tar.gz chromium_src-0a2a0e15f6741225fbce75cfc4fa07db0a00190d.tar.bz2 |
Fix handling of escaped slashes ("%2f") in FTP paths.
Chrome was unescaping them before sending requests to the server, which
would map "ftp://foo/bar%2f..%2fblah" to requesting path "/bar/../blah"
from the FTP server, which is weird. It now requests the file named
"bar%2f..%2fblah" instead. Worth noting that GURL considers this to be
a separate url from ftp://foo/blah, so its best to be consistent with
that behavior. Also, when Chrome sees an FTP file with %'s in its name,
it unescapes them, so Chrome would never generate a URL with "%2f" in
it, but they could still come from external sources.
This CL also fixes the title of directory listings of paths with "%2f"
in them - ftp://foo/bar%2f/baz will now say "Index of /bar%2f/" instead
of "Index of /bar//".
BUG=597056
Review URL: https://codereview.chromium.org/1827893002
Cr-Commit-Position: refs/heads/master@{#382961}
Diffstat (limited to 'ash/display/display_color_manager_chromeos.cc')
0 files changed, 0 insertions, 0 deletions