Bug 28885 - Invalid parsing of backslash in url function
Summary: Invalid parsing of backslash in url function
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 37641
  Show dependency treegraph
 
Reported: 2009-09-01 12:16 PDT by Erik Arvidsson
Modified: 2017-05-14 20:51 PDT (History)
5 users (show)

See Also:


Attachments
test example - worksforme (673 bytes, application/xhtml+xml)
2012-09-27 20:47 PDT, Glenn Adams
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Arvidsson 2009-09-01 12:16:41 PDT
According to http://www.w3.org/TR/css3-values/#uris

Parentheses, commas, whitespace characters, single quotes (') and double quotes (") appearing in a URI must be escaped with a backslash: '\(', '\)', '\,'.

We unescape those escaped characters but the unescaping of \\ at the end of the url is broken.

The following works

url("http://digg.com/?\\abc")

but this does not work

url("http://digg.com/?\\")

it never even tries to do a network request.
Comment 1 Alexey Proskuryakov 2010-02-16 19:08:32 PST
See also: bug 18604.
Comment 2 Glenn Adams 2012-09-27 20:01:42 PDT
(In reply to comment #1)
> See also: bug 18604.

I closed bug 18604 as invalid.
Comment 3 Glenn Adams 2012-09-27 20:47:52 PDT
Created attachment 166135 [details]
test example - worksforme
Comment 4 Glenn Adams 2012-09-27 20:50:29 PDT
the just attached test shows there is no problem with escaped backslash at end of url(), both in quoted and unquoted forms; verified (manually) that both perform network access on URL with single backslash at end (after unescape)
Comment 5 Glenn Adams 2012-09-27 20:56:49 PDT
(In reply to comment #4)
> the just attached test shows there is no problem with escaped backslash at end of url(), both in quoted and unquoted forms; verified (manually) that both perform network access on URL with single backslash at end (after unescape)

tested in 537+ mac-mountainlion