SQL Injection in CubeCart PHP Free & Commercial Shopping Cart Application 1. *Advisory Information* Title: SQL Injection in CubeCart PHP Free & Commercial Shopping Cart Application Advisory Id: CORE-2010-0415 Advisory URL: [http://www.coresecurity.com/content/cubecart-php-shopping-cart-sql-injection] Date published: 2010-06-08 Date of last update: 2010-06-08 Vendors contacted: CubeCart Release mode: Coordinated release 2. *Vulnerability Information* Class: SQL injection [CWE-89] Impact: Code execution Remotely Exploitable: Yes Locally Exploitable: No CVE Name: CVE-2010-1931 Bugtraq ID: N/A 3. *Vulnerability Description* There is an SQL Injection[1] vulnerability in the CubeCart PHP Shopping cart[2], this vulnerability may be exploited by 'HTTP POST'ing mailicious data to the index.php script of CubeCart. As an example, exploitation may result in leak of sensitive information or injection of mailicious code into the shopping cart's web page. 4. *Vulnerable packages* . CubeCart v.4.3.4 . CubeCart v.4.3.5 . CubeCart v.4.3.6 . CubeCart v.4.3.7 . CubeCart v.4.3.8 . CubeCart v.4.3.9 . Older versions are probably affected too, but they were not checked. 5. *Non-vulnerable packages* . CubeCart v.4.4.0 6. *Vendor Information, Solutions and Workarounds* Upgrade to the latest version of CubeCart, available at CubeCart's web page[1]. 7. *Credits* This vulnerability was discovered and researched by 7Safe [http://www.7safe.com/]. This advisory was coordinated by Pedro Varangot [http://corelabs.coresecurity.com/index.php?module=Wiki&action=view&type=researcher&name=Pedro_Varangot]. 8. *Technical Description / Proof of Concept Code* The shipKey parameter of the following POST request (Updating basket) is not adequately sanitized resulting in SQL Injection. /----- POST /CubeCart/index.php?_g=co&_a=step2 HTTP/1.1 Cookie: [...] Content-Type: application/x-www-form-urlencoded [...] quan%5B3afcdbfeb6ecfbdd0ba628696e3cc163%5D=3&shipKey=1'&coupon= - -----/ This happens because the 'shipKey' is assumed to be of the 'int' datatype, and is not cast nor checked before sent to the database, as we can see in the folling code snippet: /----- if(isset($_POST['shipKey']) && $_POST['shipKey']>0) { $cart->setVar($_POST['shipKey'],'shipKey'); // lose post vars $refresh = true; } - -----/ The following snippet correctly patches this bug: /----- if(isset($_POST['shipKey']) && (int)$_POST['shipKey']>0) { $cart->setVar((int)$_POST['shipKey'],'shipKey'); // lose post vars $refresh = true; } - -----/ 9. *Report Timeline* . 2010-06-02: Core Security Technologies contacts the CubeCart team informing an SQL injection vulnerability has been found, and than an advisory draft is ready. . 2010-06-02: The CubeCart team asks Core for a technical description of the vulnerability, in the form of an advisory draft, over plaintext e-mail. . 2010-06-02: Technical details are sent by Core in the form of a draft of this advisory. . 2010-06-03: The CubeCart team replies informing that CubeCart 4.4.0 will be released very soon, and that this vulnerability will be patched in that version. . 2010-06-04: CubeCart 4.4.0 is released. . 2010-06-08: Advisory CORE-2010-0415 is published.