<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>27189</bug_id>
          
          <creation_ts>2009-07-11 22:26:41 -0700</creation_ts>
          <short_desc>r45752+ nightly: @import css generates wrong path</short_desc>
          <delta_ts>2009-07-12 14:47:14 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>CSS</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://forum.dvdtalk.com/forum-feedback-support-4/</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Regression, XSSAuditor</keywords>
          <priority>P2</priority>
          <bug_severity>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Kevin M. Dean">kevin</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>dbates</cc>
    
    <cc>john</cc>
    
    <cc>mjs</cc>
    
    <cc>simon</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>131163</commentid>
    <comment_count>0</comment_count>
    <who name="Kevin M. Dean">kevin</who>
    <bug_when>2009-07-11 22:26:41 -0700</bug_when>
    <thetext>The forum&apos;s on the site load a css from:

@import url(&quot;clientscript/vbulletin_css/style-bbed93be-00019.css&quot;);

r45752+ generates a file not found and the css doesn&apos;t render when it tries to load the path:

http://forum.dvdtalk.com/forum-feedback-support-4/clientscript/vbulletin_css/style-bbed93be-00019.css

r45702 and earlier does render correctly and looks for the path:

http://forum.dvdtalk.com/clientscript/vbulletin_css/style-bbed93be-00019.css</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131181</commentid>
    <comment_count>1</comment_count>
    <who name="Mark Rowe (bdash)">mrowe</who>
    <bug_when>2009-07-12 10:52:34 -0700</bug_when>
    <thetext>Sounds like it could be another XSS auditor issue:
&gt; Refused to execute a JavaScript script. Source code of script found within request

It&apos;d be great if these errors mentioned the URL that they relate to.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131186</commentid>
    <comment_count>2</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2009-07-12 11:46:45 -0700</bug_when>
    <thetext>This issue is triggered because of the HTML Base element:
&lt;base href=&quot;http://forum.dvdtalk.com/&quot; /&gt;

XSSAuditor thinks this is an attack because the URL of the Base element appears in the URL of the page (say http://forum.dvdtalk.com/dvd-talk-3/). A check in XSSAuditor::canSetBaseElementURL (line: m_frame-&gt;document()-&gt;url().baseAsString() != baseElementURL.baseAsString()) is insufficient. Working on patch.

(In reply to comment #0)
&gt; The forum&apos;s on the site load a css from:
&gt; 
&gt; @import url(&quot;clientscript/vbulletin_css/style-bbed93be-00019.css&quot;);
&gt; 
&gt; r45752+ generates a file not found and the css doesn&apos;t render when it tries to
&gt; load the path:
&gt; 
&gt; http://forum.dvdtalk.com/forum-feedback-support-4/clientscript/vbulletin_css/style-bbed93be-00019.css
&gt; 
&gt; r45702 and earlier does render correctly and looks for the path:
&gt; 
&gt; http://forum.dvdtalk.com/clientscript/vbulletin_css/style-bbed93be-00019.css</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131190</commentid>
    <comment_count>3</comment_count>
      <attachid>32631</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2009-07-12 12:36:10 -0700</bug_when>
    <thetext>Created attachment 32631
Patch with test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131194</commentid>
    <comment_count>4</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2009-07-12 13:13:48 -0700</bug_when>
    <thetext>*** Bug 27185 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131204</commentid>
    <comment_count>5</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2009-07-12 14:45:30 -0700</bug_when>
    <thetext>*** Bug 27194 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131205</commentid>
    <comment_count>6</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2009-07-12 14:46:54 -0700</bug_when>
    <thetext>Sending        LayoutTests/ChangeLog
Adding         LayoutTests/http/tests/security/xssAuditor/base-href-safe3-expected.txt
Adding         LayoutTests/http/tests/security/xssAuditor/base-href-safe3.html
Adding         LayoutTests/http/tests/security/xssAuditor/resources/base-href/base-href-safe3.html
Sending        WebCore/ChangeLog
Sending        WebCore/page/XSSAuditor.cpp
Transmitting file data ......
Committed revision 45763.
http://trac.webkit.org/changeset/45763</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>32631</attachid>
            <date>2009-07-12 12:36:10 -0700</date>
            <delta_ts>2009-07-12 13:21:01 -0700</delta_ts>
            <desc>Patch with test</desc>
            <filename>Bug27189_1.patch</filename>
            <type>text/plain</type>
            <size>4301</size>
            <attacher name="Daniel Bates">dbates</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0NTc2MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMjAgQEAKKzIwMDktMDctMTIgIERhbmllbCBCYXRlcyAgPGRiYXRlc0BpbnR1ZGF0
YS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI3MTg5CisgICAgICAgIAor
ICAgICAgICBGaXhlcyBpbnN1ZmZpY2llbnQgY2hlY2sgaW4gWFNTQXVkaXRvcjo6Y2FuU2V0QmFz
ZUVsZW1lbnRVUkwgdGhhdCBjYXVzZWQgCisgICAgICAgIFhTU0F1ZGl0b3IgdG8gaW5jb3JyZWN0
bHkgYmxvY2sgSFRNTCBCYXNlIGVsZW1lbnRzIHdob3NlIGJhc2UgcGF0aCBjb2luY2lkZWQgCisg
ICAgICAgIHdpdGggdGhlIFVSTCBvZiB0aGUgcGFnZS4KKworICAgICAgICBUZXN0OiBodHRwL3Rl
c3RzL3NlY3VyaXR5L3hzc0F1ZGl0b3IvYmFzZS1ocmVmLXNhZmUzLmh0bWwKKworICAgICAgICAq
IHBhZ2UvWFNTQXVkaXRvci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpYU1NBdWRpdG9yOjpjYW5T
ZXRCYXNlRWxlbWVudFVSTCk6IENoYW5nZWQgY29uZGl0aW9uYWwgdG8gb25seSBjYWxsIAorICAg
ICAgICBYU1NBdWRpdG9yOjpmaW5kSW5SZXF1ZXN0KCkgaWYgdGhlIGhvc3QgaW4gdGhlIHBhZ2Ug
VVJMIGRpc2FncmVlcyB3aXRoIHRoZSBob3N0IAorICAgICAgICBpbiB0aGUgYmFzZSBlbGVtZW50
IFVSTC4KKwogMjAwOS0wNy0xMiAgWGFuIExvcGV6ICA8eGxvcGV6QGlnYWxpYS5jb20+CiAKICAg
ICAgICAgUmV2aWV3ZWQgYnkgR3VzdGF2byBOb3JvbmhhLgpJbmRleDogV2ViQ29yZS9wYWdlL1hT
U0F1ZGl0b3IuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvcGFnZS9YU1NBdWRpdG9yLmNwcAko
cmV2aXNpb24gNDU3NTcpCisrKyBXZWJDb3JlL3BhZ2UvWFNTQXVkaXRvci5jcHAJKHdvcmtpbmcg
Y29weSkKQEAgLTEzNyw3ICsxMzcsNyBAQCBib29sIFhTU0F1ZGl0b3I6OmNhblNldEJhc2VFbGVt
ZW50VVJMKGNvCiAgICAgICAgIHJldHVybiB0cnVlOwogICAgIAogICAgIEtVUkwgYmFzZUVsZW1l
bnRVUkwobV9mcmFtZS0+ZG9jdW1lbnQoKS0+dXJsKCksIHVybCk7Ci0gICAgaWYgKG1fZnJhbWUt
PmRvY3VtZW50KCktPnVybCgpLmJhc2VBc1N0cmluZygpICE9IGJhc2VFbGVtZW50VVJMLmJhc2VB
c1N0cmluZygpICYmIGZpbmRJblJlcXVlc3QodXJsKSkgeworICAgIGlmIChtX2ZyYW1lLT5kb2N1
bWVudCgpLT51cmwoKS5ob3N0KCkgIT0gYmFzZUVsZW1lbnRVUkwuaG9zdCgpICYmIGZpbmRJblJl
cXVlc3QodXJsKSkgewogICAgICAgICBERUZJTkVfU1RBVElDX0xPQ0FMKFN0cmluZywgY29uc29s
ZU1lc3NhZ2UsICgiUmVmdXNlZCB0byBleGVjdXRlIGEgSmF2YVNjcmlwdCBzY3JpcHQuIFNvdXJj
ZSBjb2RlIG9mIHNjcmlwdCBmb3VuZCB3aXRoaW4gcmVxdWVzdCIpKTsKICAgICAgICAgbV9mcmFt
ZS0+ZG9tV2luZG93KCktPmNvbnNvbGUoKS0+YWRkTWVzc2FnZShKU01lc3NhZ2VTb3VyY2UsIEVy
cm9yTWVzc2FnZUxldmVsLCBjb25zb2xlTWVzc2FnZSwgMSwgU3RyaW5nKCkpOwogICAgICAgICBy
ZXR1cm4gZmFsc2U7CkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g
TGF5b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZpc2lvbiA0NTc2MSkKKysrIExheW91dFRlc3RzL0No
YW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDA5LTA3LTEyICBEYW5p
ZWwgQmF0ZXMgIDxkYmF0ZXNAaW50dWRhdGEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD0yNzE4OQorICAgICAgICAKKyAgICAgICAgVGVzdHMgdGhhdCBYU1NBdWRpdG9yIGRv
ZXMgbm90IGJsb2NrIEhUTUwgQmFzZSBlbGVtZW50cyB3aG9zZSBwYXRoIGhhcyB0aGUgCisgICAg
ICAgIHNhbWUgaG9zdCBhcyB0aGUgcGFnZS4KKworICAgICAgICAqIGh0dHAvdGVzdHMvc2VjdXJp
dHkveHNzQXVkaXRvci9iYXNlLWhyZWYtc2FmZTMtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAg
ICAgKiBodHRwL3Rlc3RzL3NlY3VyaXR5L3hzc0F1ZGl0b3IvYmFzZS1ocmVmLXNhZmUzLmh0bWw6
IEFkZGVkLgorICAgICAgICAqIGh0dHAvdGVzdHMvc2VjdXJpdHkveHNzQXVkaXRvci9yZXNvdXJj
ZXMvYmFzZS1ocmVmL2Jhc2UtaHJlZi1zYWZlMy5odG1sOiBBZGRlZC4KKwogMjAwOS0wNy0xMSAg
T2xpdmVyIEh1bnQgIDxvbGl2ZXJAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFNp
bW9uIEZyYXNlci4KSW5kZXg6IExheW91dFRlc3RzL2h0dHAvdGVzdHMvc2VjdXJpdHkveHNzQXVk
aXRvci9iYXNlLWhyZWYtc2FmZTMtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRl
c3RzL2h0dHAvdGVzdHMvc2VjdXJpdHkveHNzQXVkaXRvci9iYXNlLWhyZWYtc2FmZTMtZXhwZWN0
ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9zZWN1cml0eS94
c3NBdWRpdG9yL2Jhc2UtaHJlZi1zYWZlMy1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCkBAIC0w
LDAgKzEsMiBAQAorQUxFUlQ6IFRoaXMgaXMgYSBzYWZlIHNjcmlwdC4KKwpJbmRleDogTGF5b3V0
VGVzdHMvaHR0cC90ZXN0cy9zZWN1cml0eS94c3NBdWRpdG9yL2Jhc2UtaHJlZi1zYWZlMy5odG1s
Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2h0dHAvdGVzdHMvc2VjdXJpdHkveHNzQXVkaXRv
ci9iYXNlLWhyZWYtc2FmZTMuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2h0dHAv
dGVzdHMvc2VjdXJpdHkveHNzQXVkaXRvci9iYXNlLWhyZWYtc2FmZTMuaHRtbAkocmV2aXNpb24g
MCkKQEAgLTAsMCArMSwxNSBAQAorPCFET0NUWVBFIGh0bWw+Cis8aHRtbD4KKzxoZWFkPgorPHNj
cmlwdD4KK2lmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpIHsKKyAgbGF5b3V0VGVzdENv
bnRyb2xsZXIuZHVtcEFzVGV4dCgpOworICBsYXlvdXRUZXN0Q29udHJvbGxlci5zZXRYU1NBdWRp
dG9yRW5hYmxlZCh0cnVlKTsKK30KKzwvc2NyaXB0PgorPC9oZWFkPgorPGJvZHk+Cis8aWZyYW1l
IHNyYz0iaHR0cDovL2xvY2FsaG9zdDo4MDAwL3NlY3VyaXR5L3hzc0F1ZGl0b3IvcmVzb3VyY2Vz
L2Jhc2UtaHJlZi9iYXNlLWhyZWYtc2FmZTMuaHRtbCI+Cis8L2lmcmFtZT4KKzwvYm9keT4KKzwv
aHRtbD4KSW5kZXg6IExheW91dFRlc3RzL2h0dHAvdGVzdHMvc2VjdXJpdHkveHNzQXVkaXRvci9y
ZXNvdXJjZXMvYmFzZS1ocmVmL2Jhc2UtaHJlZi1zYWZlMy5odG1sCj09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExh
eW91dFRlc3RzL2h0dHAvdGVzdHMvc2VjdXJpdHkveHNzQXVkaXRvci9yZXNvdXJjZXMvYmFzZS1o
cmVmL2Jhc2UtaHJlZi1zYWZlMy5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvaHR0
cC90ZXN0cy9zZWN1cml0eS94c3NBdWRpdG9yL3Jlc291cmNlcy9iYXNlLWhyZWYvYmFzZS1ocmVm
LXNhZmUzLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTUgQEAKKzwhRE9DVFlQRSBodG1s
PgorPGh0bWw+Cis8aGVhZD4KKzxzY3JpcHQ+CitpZiAod2luZG93LmxheW91dFRlc3RDb250cm9s
bGVyKSB7CisgIGxheW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsKKyAgbGF5b3V0VGVz
dENvbnRyb2xsZXIuc2V0WFNTQXVkaXRvckVuYWJsZWQodHJ1ZSk7Cit9Cis8L3NjcmlwdD4KKzxi
YXNlIGhyZWY9J2h0dHA6Ly9sb2NhbGhvc3Q6ODAwMC9zZWN1cml0eS94c3NBdWRpdG9yL3Jlc291
cmNlcy8nPgorPC9oZWFkPgorPGJvZHk+Cis8c2NyaXB0IHNyYz0ic2FmZS1zY3JpcHQuanMiPjwv
c2NyaXB0PgorPC9ib2R5PgorPC9odG1sPgo=
</data>
<flag name="review"
          id="17033"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>