Last modified: 2014-01-14 07:24:32 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T60472, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58472 - Opera 12 -o-link XSS (user interaction required)
Opera 12 -o-link XSS (user interaction required)
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Parser (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 59830
  Show dependency treegraph
 
Reported: 2013-12-13 22:46 UTC by Chris Steipp
Modified: 2014-01-14 07:24 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Disallow -o-link in styles (2.05 KB, patch)
2013-12-14 01:15 UTC, Chris Steipp
Details
Disallow -o-link in styles (2.07 KB, patch)
2013-12-16 19:08 UTC, Chris Steipp
Details
Disallow -o-link in styles (1.19 branch) (2.09 KB, patch)
2014-01-08 23:09 UTC, Markus Glaser
Details
Disallow -o-link in styles (1.21 branch) (2.08 KB, patch)
2014-01-08 23:09 UTC, Markus Glaser
Details
Disallow -o-link in styles (1.22 branch) (2.01 KB, patch)
2014-01-08 23:10 UTC, Markus Glaser
Details

Description Chris Steipp 2013-12-13 22:46:40 UTC
Adding this to a page will execute javascript when clicked. This is a variant of http://html5sec.org/#9. 

<div title="&#100;&#97;&#116;&#97;&#58;&#116;&#101;&#120;&#116;&#47;&#104;&#116;&#109;&#108;&#44;&#60;&#105;&#109;&#103;&#32;&#115;&#114;&#99;&#61;&#49;&#32;&#111;&#110;&#101;&#114;&#114;&#111;&#114;&#61;&#97;&#108;&#101;&#114;&#116;&#40;&#49;&#41;&#62;" style="-o-link:attr(title);-o-link-source:current">Click Me</div>


I can't see any good reason to allow -o-link in the style, since most browsers ignore it.
Comment 1 Chris Steipp 2013-12-14 01:15:55 UTC
Created attachment 14095 [details]
Disallow -o-link in styles
Comment 2 Tim Starling 2013-12-16 03:23:16 UTC
So you have verified this in a recent version of Opera? The link extensions have been removed from the Opera documentation, but you can find them in IA:

<https://web.archive.org/web/20030602071146/http://www.opera.com/docs/specs/#xml-css-link>

opera.com still hosts documentation for Opera 7, which was the current version at the time of that archive, but it seems they later decided to edit out all mention of that feature.

Maybe if -o-link still exists, then -o-replace may also still exist? It should probably be blacklisted also.
Comment 3 Chris Steipp 2013-12-16 18:49:20 UTC
I verified it with 12.15, it looks like 12.16 is the most current.

-o-replace gives me an unknown property error, but probably best to blacklist it too, just in case.
Comment 4 Chris Steipp 2013-12-16 19:08:07 UTC
Created attachment 14109 [details]
Disallow -o-link in styles

Forbid -o-replace too
Comment 5 Krinkle 2013-12-16 21:03:33 UTC
Well, Opera 12.16 is the most current, ... of the Presto branch (which is no longer advertised).

Since early 2013, Opera has been reimplemented using Chromium (Blink, V8, etc.). First beta (Opera 15, version 13/14 were skipped) in May 2013, and gone stable since. They're up to Opera 18 already, and do have auto-updating.

However Opera 12 does not auto-update to Opera >= 15, so Opera 12 continues to have a fair browser share for now (seems good to support, especially when relatively inexpensive and in the interest of security).
Comment 6 Tim Starling 2013-12-16 23:02:56 UTC
Looks good.
Comment 7 Chris Steipp 2013-12-23 18:28:31 UTC
This has been assigned CVE-2013-6454
Comment 8 Markus Glaser 2014-01-08 23:09:14 UTC
Created attachment 14264 [details]
Disallow -o-link in styles (1.19 branch)
Comment 9 Markus Glaser 2014-01-08 23:09:52 UTC
Created attachment 14265 [details]
Disallow -o-link in styles (1.21 branch)
Comment 10 Markus Glaser 2014-01-08 23:10:23 UTC
Created attachment 14266 [details]
Disallow -o-link in styles (1.22 branch)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links