SA-CONTRIB-2012-086 - Amadou - Cross Site Scripting

The Amadou theme outputs additional first and last classes to the list of links to help out themers. This was being done in a way that was not secure.

A Cross Site Scripting (XSS) vulnerability was identified in Amadou theme’s themes_links() function in the template.php file, which was fixed in the theme_links() function in Drupal 6.3 as noted in (SA-2008-044 <http://drupal.org/node/280571&gt;).

CVE: CVE-2012-2715

Versions affected

• Amadou 6.x-1.x versions prior to 6.x-1.3.

Drupal core is not affected. If you do not use the contributed Amadou theme, there is nothing you need to do.

Solution

Install the latest version:

• If you use the Amadou theme for Drupal 6.x, upgrade to Amadou 6.x-1.3.

If you have created a custom theme with Amadou or are using Amadou as a base theme, please see below for a manual fix:

The offending code on line 77 in the original template.php file:
$output .= '<span>';

Should be replaced with the following code:
$output .= '<li> $class)) .'&gt;';

Also see the Amadou project page.

Reported by

• Peter Wolanin and Matt Chapman of the Drupal Security Team

Fixed by

• Jason Moore, Amadou theme maintainer
• Matt Chapman of the Drupal Security Team

Coordinated by

• Matt Chapman of the Drupal Security Team