CubeCart PHP (shipkey parameter) <= 4.3.x Remote SQL Inje
Posted on 29 June 2010
<!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN'><html><head><meta http-equiv='Content-Type' content='text/html; charset=windows-1251'><title>CubeCart PHP (shipkey parameter) <= 4.3.x Remote SQL Injection</title><link rel='shortcut icon' href='/favicon.ico' type='image/x-icon'><link rel='alternate' type='application/rss+xml' title='Inj3ct0r RSS' href='/rss'></head><body><pre>============================================================== CubeCart PHP (shipkey parameter) <= 4.3.x Remote SQL Injection ============================================================== 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; } - -----/ # <a href='http://inj3ct0r.com/'>Inj3ct0r.com</a> [2010-06-29]</pre><script type='text/javascript'>var gaJsHost = (("https:" == document.location.protocol) ? "https://ssl." : "http://www.");document.write(unescape("%3Cscript src='" + gaJsHost + "google-analytics.com/ga.js' type='text/javascript'%3E%3C/script%3E"));</script><script type='text/javascript'>try{var pageTracker = _gat._getTracker("UA-12725838-1");pageTracker._setDomainName("none");pageTracker._setAllowLinker(true);pageTracker._trackPageview();}catch(err){}</script></body></html>