oss-sec mailing list archives

SQL Injection Vulnerability in Ruby on Rails (CVE-2012-2661)


From: Aaron Patterson <tenderlove () ruby-lang org>
Date: Thu, 31 May 2012 12:16:56 -0700

SQL Injection Vulnerability in Ruby on Rails

There is a SQL injection vulnerability in Active Record, version 3.0 and later. This vulnerability has been assigned 
the CVE identifier CVE-2012-2661.

Versions Affected:  3.0.0 and ALL later versions
Not affected:       2.3.14
Fixed Versions:     3.2.4, 3.1.5, 3.0.13

Impact 
------ 
Due to the way Active Record handles nested query parameters, an attacker can use a specially crafted request to inject 
some forms of SQL into your application's SQL queries.

All users running an affected release should upgrade immediately.

Impacted code directly passes request params to the `where` method of an ActiveRecord class like this:

    Post.where(:id => params[:id]).all

An attacker can make a request that causes `params[:id]` to return a specially crafted hash that will cause the WHERE 
clause of the SQL statement to query an arbitrary table with some value.

Releases 
-------- 
The FIXED releases are available at the normal locations. 

Workarounds 
----------- 
This issue can be mitigated by casting the parameter to an expected value.  For example, change this:

    Post.where(:id => params[:id]).all

to this:

    Post.where(:id => params[:id].to_s).all

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.  We have also provided a patch for the 3.0 series despite 
the fact it is unmaintained.

* 3-0-params_sql_injection.patch - Patch for 3.0 series 
* 3-1-params_sql_injection.patch - Patch for 3.1 series 
* 3-2-params_sql_injection.patch - Patch for 3.2 series 

Please note that only the  3.1.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 Ben Murphy for reporting the vulnerability to us, and to Chad Pyne of thoughtbot for helping us verify the 
fix.

-- 
Aaron Patterson
http://tenderlovemaking.com/

Attachment: 3-0-params_sql_injection.patch
Description:

Attachment: 3-1-params_sql_injection.patch
Description:

Attachment: 3-2-params_sql_injection.patch
Description:

Attachment: _bin
Description:


Current thread: