CubeCart PHP <=4.3.x Remote SQL注入漏洞

[+]info:
~~~~~~~~~
QL 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].

[+]poc:
~~~~~~~~~
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;
}
- -----/

[+]Reference:
~~~~~~~~~
http://www.exploit-db.com/exploits/14117