New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 808525 link

Starred by 3 users

Issue metadata

Status: Duplicate
Merged: issue 807304
Owner:
Buried. Ping if important.
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 2
Type: Bug



Sign in to add a comment

XSS report URL error message incorrect

Reported by q...@yelp.com, Feb 2 2018

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.94 Safari/537.36

Steps to reproduce the problem:
1. Serve a X-XSS-Protection header with the reporting URL with a different origin
2. Observe the error "insecure reporting URL for secure page at character position 10. The default protections will be applied."

What is the expected behavior?
The error should have been "reporting URL is not same scheme, host, and port as page"

What went wrong?
https://chromium-review.googlesource.com/c/chromium/src/+/768367/3/third_party/WebKit/Source/core/html/parser/XSSAuditor.cpp introduced a check for whether the report URL has the same origin as the request. The added code will blank out xss_protection_report_url. However, since the IsMixedContent check immediately afterwards is using xss_protection_report_url, it now reports it as mixed content, even if both origins are HTTPS.

I believe the fix is to (optionally switch the order of these two checks and) put the second check behind an else, so that they will not trigger each other.

Did this work before? N/A 

Chrome version: 64  Channel: n/a
OS Version: OS X 10.12.6
Flash Version:
 
Components: Blink>SecurityFeature>XSSAuditor
Labels: -Type-Bug-Security -Restrict-View-SecurityTeam Type-Bug
Owner: mkwst@chromium.org
We can treat this as a functional bug since it does not provide access to any information to which the origin is not entitled, just a message error.
Cc: tsepez@chromium.org jochen@chromium.org
Labels: OS-Android OS-Chrome OS-Fuchsia OS-Linux OS-Windows
Status: Assigned (was: Unconfirmed)
Mergedinto: 807304
Status: Duplicate (was: Assigned)

Sign in to add a comment