Closed Bug 1254226 Opened 10 years ago Closed 10 years ago

XSS through javascript: callback URLs in auth delegation

Categories

(Bugzilla :: User Accounts, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 6.0

People

(Reporter: jwkbugzilla, Assigned: LpSolit)

References

Details

(Keywords: reporter-external, sec-moderate, wsec-xss, Whiteboard: XSS if user clicks on explicit javascript: link on confirmation page)

Attachments

(1 file)

This issue can be reproduced if the Bugzilla instance has auth delegation enabled, consider the following link for example: > https://bugzilla.mozilla.org/auth.cgi?callback=javascript:alert%28/xss/%29&description=foobar Clicking the link will produce a message saying "xss." The issue is not too critical because the link title gives it away and obfuscation possibilities are limited, still a vulnerability however. Vanilla Bugzilla instances are affected as well, landfill.bugzilla.org doesn't have auth.cgi however. Note that auth.cgi only allows HTTP and HTTPS as callbacks, the corresponding check only happens when confirming delegation however.
Hum, where do you see the "xss" message? Clicking this link generates a page with: A third-party website (javascript:alert(/xss/)) would like to have complete access to your Bugzilla account. The description of the site reads: foobar Do you want this website to have complete access to your Bugzilla account? But no alert() is triggered.
The alert is produced if you click the website link on this page.
Attached patch patch, v1Splinter Review
Assignee: user-accounts → LpSolit
Status: NEW → ASSIGNED
Attachment #8727580 - Flags: review?(dkl)
Severity: normal → minor
Target Milestone: --- → Bugzilla 6.0
Depends on: 1144468
Flags: sec-bounty?
Comment on attachment 8727580 [details] [diff] [review] patch, v1 Review of attachment 8727580 [details] [diff] [review]: ----------------------------------------------------------------- r=dylan
Attachment #8727580 - Flags: review?(dkl) → review+
Why is this bug tagged sec-high? Not only must the victim first load a page such as the one described in comment 0, but then the victim must also click the URL displayed in that page to trigger XSS. So it requires two actions from the user. Being an authentication page, I doubt one would click on a link asking for full access to your account when the website name is "javascript:alert(/xss/)". XSS is not triggered when loading the initial page; only when clicking on the "javascript:alert(/xss/)" link. FYI, there is no Bugzilla release with that code yet as it's in master only.
(In reply to Frédéric Buclin from comment #5) > FYI, there is no Bugzilla release with that code yet as it's in master only. Seems to be correct but BMO is using this already. The rest - not up to me.
Keywords: sec-highsec-moderate
Whiteboard: XSS if user clicks on explicit javascript: link on confirmation page
To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git c7e8031..3368986 master -> master
Group: bugzilla-security
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Flags: sec-bounty? → sec-bounty+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: