Closed (fixed)
Project:
Google Plus One Button | Google+ Badge
Version:
6.x-1.1
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Feb 2012 at 14:03 UTC
Updated:
1 Jun 2012 at 09:10 UTC
The node type labels in admin interface don't santize. A node type with label <script>alert('XSS');</script> result a JS popup.
Change 34 line in google_plusone.admin.inc
'#options' => node_get_types('names'),
to
'#options' => array_map('check_plain', node_get_types('names')),
Comments
Comment #1
corbacho commentedThanks Plazik
I was searching in d.o, and saw that Flag module had this bug also in the past and was considered bug, not security issue #483218: role names are not properly sanitized by flag module, since accessing to this page requires administration permission.
Anyway, I will fix it soon, probably tomorrow.
Comment #2
corbacho commentedMarking now as fixed.
It has been for several months in the 7.x branch but finally I push the change also to 6.x branch (dev), Sorry for the delay