Last modified: 2014-09-30 21:57:15 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 T53521, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 51521 - VisualEditor: Broken OpenODBC browser plugin causes "application/iodbc" to be injected into the page
VisualEditor: Broken OpenODBC browser plugin causes "application/iodbc" to be...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: Low minor
: VE-deploy-2014-10-02
Assigned To: Alex Monk
:
Depends on:
Blocks: ve-browserplugin
  Show dependency treegraph
 
Reported: 2013-07-17 10:38 UTC by Maury Markowitz
Modified: 2014-09-30 21:57 UTC (History)
12 users (show)

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


Attachments

Description Maury Markowitz 2013-07-17 10:38:09 UTC
I had OpenODBC's iODBC plugin installed in Safari (latest release, always). After any edit, EMBED tags would be inserted at the top and bottom of the article. This did not occur with the source editor, or any other editable widget, like this one I'm typing in now.
Comment 1 PiRSquared17 2013-07-17 16:26:48 UTC
Have you reported this upstream (i.e., to the author of the extension)? I don't this it's VE's fault, although I've seen similar stuff with another extension (FoxLingo):
[[m:en:Wikipedia:VisualEditor/Feedback/Archive_2013_07#Inserting_weird_script_tag_stuff]].

Actually, might be a more general bug, as these diffs have many similarities. I wonder if any other browser extensions do this.
Comment 2 Maury Markowitz 2013-07-17 18:40:22 UTC
Actually my post over on the Village Pump *appears* to have turned up several other examples of plugins having the same problem. Not much more I can add though...

http://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)

Look for "Is this me or the editor?" (I can't get the section link to work)
Comment 3 PiRSquared17 2013-07-18 16:40:51 UTC
Have you reported it to the author yet?
Comment 4 John Mark Vandenberg 2013-07-21 23:14:32 UTC
It is adding
<embed type="application/iodbc" width="0" height="0" />
Comment 5 Yellowcard 2013-07-29 20:02:34 UTC
From my understanding, using this bug targetedly it is possible for Browser Add-Ons to insert malicious code in Wikipedia pages. The Wikipedia user will not even notice. I put importance to "immediate", in case I'm wrong please undo my change.
Comment 6 John Mark Vandenberg 2013-07-29 20:57:20 UTC
(In reply to comment #5)
> From my understanding, using this bug targetedly it is possible for Browser
> Add-Ons to insert malicious code in Wikipedia pages. The Wikipedia user will
> not even notice. I put importance to "immediate", in case I'm wrong please
> undo
> my change.

embed is not one of the allowed html tags when $wgRawHtml is disabled, so the embed tag is escaped as a security measure.  No malicious injection is possible via this bug.

The now archived discussion is
https://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28technical%29/Archive_114#Is_this_me_or_the_editor.3F

Examples:
https://en.wikipedia.org/w/index.php?title=The_Road_%282009_film%29&diff=prev&oldid=563739949
https://en.wikipedia.org/w/index.php?title=The_Road_%282009_film%29&diff=prev&oldid=564586766
https://en.wikipedia.org/w/index.php?title=Informix_Wingz&diff=prev&oldid=564587579
Comment 7 PiRSquared17 2013-07-31 03:16:57 UTC
You could write an AbuseFilter to prevent edits like this. This is not VE's fault. There is a problem with the extension. Has anyone notified the author of said extension?
Comment 8 Maury Markowitz 2013-07-31 12:41:02 UTC
"This is not VE's fault."

I am not at all certain of this. This problem ONLY occurs in VE. It does not occur in the source editor, or any other text editor across the whole wide web.

Something in VE is triggering this plugin to insert code. The plugin and the code it's inserting, is perfectly valid.
Comment 9 Alex Monk 2013-07-31 15:36:38 UTC
PiRSquared17 is right. This is not VE's fault. It only occurs in VE because of the way it sets up the editing area, which your plugin doesn't recognise as not to touch.

No and no.
Comment 10 PiRSquared17 2014-03-26 20:17:01 UTC
This is now assigned to James Forrester. Will a special test for this specific text be added? What can be done to fix this bug? Seems more like an upstream issue IMHO.
Comment 11 Andre Klapper 2014-03-26 23:34:19 UTC
(In reply to PiRSquared17 from comment #10)
> This is now assigned to James Forrester.

No it isn't. Above line clearly says:

Assigned To: 	 VE team bugs – take if you're interested!
Comment 12 Alex Monk 2014-05-17 14:16:37 UTC
Can't see a link to a bug filed upstream -> rm upstream keyword
Comment 13 Gerrit Notification Bot 2014-09-30 21:38:26 UTC
Change 163961 had a related patch set uploaded by Alex Monk:
Remove certain blacklisted elements when getting HTML from document

https://gerrit.wikimedia.org/r/163961
Comment 14 Gerrit Notification Bot 2014-09-30 21:53:20 UTC
Change 163961 merged by jenkins-bot:
Remove certain blacklisted elements when getting HTML from document

https://gerrit.wikimedia.org/r/163961

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


Navigation
Links