########################## WwW.BugReport.ir 
###########################################
#
#      AmnPardaz Security Research & Penetration Testing Group
#
# Title: Jupiter Cms Multiple Vulnerabilities
# Vendor: http://www.jupiterportal.com 
# Bugs: Local File Inclusion, Privileges Escalation
# Vulnerable Version: 1.1.5ex (prior versions also may be affected)
# Exploitation: Remote with browser
# Exploit: Available
# Fix Available: No!
###################################################################################


####################
- Description:
####################

Quote from vendor: "Jupiter is one of the most lightweight portal  
systems available and it`s open source".

####################
- Vulnerability:
####################

Improper use of extract() result in multiple vulnerability Such as LFI & PE

+-->Local File Inclusion (Remote Code Execution)

Code Snippet:

/index.php line#609-615

if(isset($n))
{
=09if(file_exists("$n.php"))
=09{
=09=09if(strpos($n, "../") !== false) header("location: $PHP_SELF?i=error");
=09=09else include("$n.php");
=09}

It's possible for an attacker to set $n variable! although we have a  
backward directory traversal check but because of index.php exists in  
the main directory of application
Attacker can upload php codes with image/gif type and include it from  
images/avatars directory!

POC: http://localhost/jupiter/index.php?n=images/avatars/aa.gif%00 

+-->Privileges Escalation

There is a logical weakness in $db->updateRow() which could result in  
privileges escalation in conjunction with extract() weakness in  
profile update process.

Code Snippet:

/include/functions_db.php line#158-174

function updateRow($table,$array,$condition)
=09{
=09=09if(count($array)==0) return;
=09=09$q="UPDATE $table SET ";
=09=09foreach($array as $index=>$value)
=09=09{
=09=09=09if($value==NULL)
=09=09=09=09$q.="`$index`=NULL, ";
=09=09=09else
=09=09=09{
=09=09=09=09$value=mysql_escape_string($value);
=09=09=09=09$q.="`$index`='$value', ";
=09=09=09}
=09=09}
=09=09$q=substr($q,0,-2)." WHERE $condition LIMIT 1";
=09=09$this->query($q);
=09}

/modules/panel.php line#328-344

=09=09$tmp['email'] = $editemail;
=09=09$tmp['url'] = $editurl;
=09=09$tmp['flag'] = $editflag;
=09=09$tmp['location'] = $editlocation;
=09=09$tmp['age'] = $editage;
=09=09$tmp['hideemail'] = $edithideemail;
=09=09$tmp['calendarbday'] = $editcalendarbday;
=09=09$tmp['msn'] = $editmsn;
=09=09$tmp['yahoo'] = $edityahoo;
=09=09$tmp['icq'] = $editicq;
=09=09$tmp['aim'] = $editaim;
=09=09$tmp['skype'] = $editskype;
=09=09$tmp['signature'] = $editsignature;
=09=09$tmp['aboutme']= $editaboutme;
=09=09$tmp['templates']= $edittemplate;

=09=09$db->updateRow("users",$tmp,"id={$user['id']}");

$tmp[authorization] which handles users access level can be set at  
this point with $_GET ,$_POST or $_COOCKIE because of improper use of  
extract();

####################
- PoC :
####################
http://www.bugreport.ir/?/23/exploit 
Original Advisory: http://www.bugreport.ir/?/23 

####################
- Credit :
####################
AmnPardaz Security Research & Penetration Testing Group
Contact: admin[4t}bugreport{d0t]ir
WwW.BugReport.ir 
WwW.AmnPardaz.com