phpmoneybooks 1.03 – Persistent Cross-Site Scripting
# Exploit Title: phpmoneybooks 1.03 Stored XSS # Date: Jun 28, 2012 # Exploit Author: chap0 - chap0.blogspot.com - @_chap0 # Vendor Homepage: http://phpmoneybooks.com/ # Software Link: http://sourceforge.net/projects/phpmoneybooks/files/phpMoneyBooks103.zip/download # Version: 1.03 # Patch: Upgrade to 1.04 Vendor Description: phpMoneyBooks is an open sourced php/mysql program. A free alternative to QuickBooks. Summary: phpmoneybooks 1.03 is vulnerable to Stored XSS vulnerability enabling an attacker to execute arbitrary JavaScript code withing the application. The vulnerability can be utilized when adding a new bank account or customer account. Users other then the admin account are able to input this information which in return can enable the super admin user to fall victim to this attack. The vulnerable index pages reside in /banks/index.php and /customers/index.php. Stored XSS example: '><script>alert('XSS')</script> Disclosure Timeline: June 28, 2012 - Contacted Vendor June 28, 2012 - Vendor replied, and patch application Vulnerable Code: /banks/index.php 40 $_POST[AcctName]=trim($_POST[AcctName]); 41 if(strtolower($row[1])==strtolower($_POST['AcctName'])) { 42 echo "<script type='text/javascript'> 43 alert('Duplicate account: $_POST[AcctName] already exists.'); 44 </script>"; 45 $_GET[action]="AddForm"; /customers/index.php 36 if($_GET[action]=="AddUser"){ 37 $query = "INSERT INTO phpMB_customers (AcctNo,DisplayName, CompanyName,MrMs,FirstName,MiddleIn,LastName,Contact,Phone,Phone2,Fax,Email,Rela tion,BillingAddress,ShippingAddress,Notes) VALUES ('$_POST[AcctNo]', '$_POST[DisplayName]', '$_POST[CompanyName]', '$_POST[MrMs]', '$_POST [FirstName]','$_POST[MiddleIn]', '$_POST[LastName]','$_POST[Contact]', '$_POST[Phone]', '$_POST[Phone2]','$_POST[FAX]','$_POST [Email]', 'Customer','$_POST[BillingAddress]', '$_POST[ShippingAddress]', '$_POST[Notes]')"; 38 QueryMysql($query); 39 $_GET[action]=""; By adding strip_tags to the strings in the php code allows the user input to be sanitized. A couple of other vulnerabilities that exist in this application: Usernames and passwords sent in clear text at log in. The users cookie gets set as username and MD5 password of the user. With this if an attacker inject javascript that steals cookies, the attacker will obtain the users username and MD5 hashed password. These two vulnerabilities are not fix, vendor was notified and is aware. |