XSS Vulnerability in number_to_currency, number_to_percentage and number_to_human (CVE-2014-0081)

46 views
Skip to first unread message

Aaron Patterson

unread,
Feb 18, 2014, 2:12:45 PM2/18/14
to ruby-sec...@googlegroups.com
XSS Vulnerability in number_to_currency, number_to_percentage and number_to_human

There is an XSS vulnerability in the number_to_currency, number_to_percentage
and number_to_human helpers in Ruby on Rails. This vulnerability has been
assigned the CVE identifier CVE-2014-0081.

Versions Affected: All.
Fixed Versions: 4.1.0.beta2, 4.0.3, 3.2.17.

Impact
------
These helpers allows users to nicely format a numeric value. Some of the parameters
to the helper (format, negative_format and units) are not escaped correctly.
Application which pass user controlled data as one of these parameters are
vulnerable to an XSS attack.

All users passing user controlled data to these parameters of the number helpers
should either upgrade or use one of the workarounds immediately.

Releases
--------
The 4.1.0.rc1, 4.0.3 and 3.2.17 releases are available at the normal locations.

Workarounds
-----------

The workaround for this issue is to escape the value passed to the parameter.
For example, replace code like this:

```
<%= number_to_currency(1.02, format: params[:format]) %>
```

With code like this

```
<%= number_to_currency(1.02, format: h(params[:format])) %>
```

Patches
-------
To aid users who aren't able to upgrade immediately we have provided patches for
the two supported release series. They are in git-am format and consist of a
single changeset.

* 4-1-beta-number_helpers_xss.patch - Patch for 4.1-beta series
* 4-0-number_helpers_xss.patch - Patch for 4.0 series
* 3-2-number_helpers_xss.patch - Patch for 3.2 series

Please note that only the 4.0.x and 3.2.x series are supported at present. Users
of earlier unsupported releases are advised to upgrade as soon as possible as we
cannot guarantee the continued availability of security fixes for unsupported
releases.

Credits
-------

Thanks to Kevin Reintjes for reporting the issue to us.

--
Aaron Patterson
http://tenderlovemaking.com/
3-2-number_helpers_xss.patch
4-0-number_helpers_xss.patch
4-1-beta-number_helpers_xss.patch
Reply all
Reply to author
Forward
0 new messages