### 简要描述:
PHPYUN最新版(phpyun_v3.1.0604_gbk)多处SQL注入及越权操作
php网站漏洞。虽然PHPYUN在注入防御上已经做得很不错了,方的很严格,像吃掉引号,宽字节的基本上很少了,但是不需要跟引号斗争的地方还有很多,得好好检查,好好修复了!!!
这里再来三处SQL注入及越权操作!!!
### 详细说明:
有sql注入漏洞的网站、文件/member/model/com.class.php
第一处SQL注入、越权修改企业环境展示信息:
```
php登陆 漏洞,function saveshow_action(){
if($_POST['submitbtn']){
$pid=@implode(',',$_POST['id']);
越权漏洞产生的原因?$company_show=$this->obj->DB_select_all("company_show","`id` in (".$pid.") and `uid`='".$this->uid."'","`id`");
if($company_show&&is_array($company_show)){
foreach($company_show as $val){
越权访问漏洞,$title=$_POST['title_'.$val['id']];
$this->obj->update_once("company_show",array("title"=>trim($title)),array("id"=>(int)$val['id']));
}
越权漏洞基本类型、$this->obj->ACT_layer_msg("修改成功!",9,"index.php?c=show");
}else{
$this->obj->ACT_layer_msg("非法操作!",3,"index.php");
}
}else{
$this->obj->ACT_msg("index.php","非法操作!");
}
}
```
这里的$pid=@implode(',',$_POST['id']);
没有经过引号保护,直接进入DB_select_all,在DB_select_all中也未进行处理:
```
function DB_select_all($tablename, $where = 1, $select = "*") {
$cachename=$tablename.$where;
if(!$row_return=$this->Memcache_set($cachename)){
$row_return=array();
$SQL = "SELECT $select FROM `" . $this->def . $tablename . "` WHERE $where";
$query=$this->db->query($SQL);
while($row=$this->db->fetch_array($query)){$row_return[]=$row;}
$this->Memcache_set($cachename,$row_return);
}
return $row_return;
}
```
导致存在SQL注入。
第二处SQL注入、越权删除企业新闻
```
function news_action(){
$this->public_action();
$where='';
if($_POST['delid'] || $_GET['delid'])
{
if($_POST['delid'] || $_GET['delid'])
{
if(is_array($_POST['delid']))
{
$delid=@implode(",",$_POST['delid']);
$layer_type='1';
}else{
$delid=$_GET['delid'];
$layer_type='0';
}
$oid=$this->obj->DB_delete_all("company_news","`id` in (".$delid.") and `uid`='".$this->uid."'","");
$oid?$this->layer_msg('删除成功!',9,$layer_type):$this->layer_msg('删除失败!',8,$layer_type);
}else{
$this->obj->ACT_layer_msg("请选择您要删除的新闻!",8,$_SERVER['HTTP_REFERER']);
}
}
```
这里的$delid=$_GET['delid'];
没有经过引号保护,直接进入了DB_delete_all中
```
function DB_delete_all($tablename, $where, $limit = 'limit 1'){
$SQL = "DELETE FROM `" . $this->def . $tablename . "` WHERE $where $limit";
$this->db->query("set `sql_mode`=''");
return $this->db->query($SQL);
}
```
导致SQL注入。
又由于,通过截断,修改后面的uid,就可以导致越权操作,删除任意企业用户的企业新闻。
第三处SQL注入、越权删除任意企业用户产品
```
function product_action()
{
$this->public_action();
$delid=$_GET['delid'];
if($delid){
if(is_array($delid)){
$ids=implode(',',$delid);
$layer_type=1;
}else{
$ids=$delid;
$layer_type=0;
}
$row=$this->obj->DB_select_all("company_product","`id` in (".$ids.") and `uid`='".$this->uid."'","`pic`");
if(is_array($row)){
foreach($row as $k=>$v){
$this->obj->unlink_pic("..".$v['pic']);
}
}
$oid=$this->obj->DB_delete_all("company_product","`id` in (".$ids.") and `uid`='".$this->uid."'","");
$oid?$this->layer_msg('删除成功!',9,$layer_type,$_SERVER['HTTP_REFERER']):$this->layer_msg('删除失败!',8,$layer_type,$_SERVER['HTTP_REFERER']);
}
```
这里的$delid=$_GET['delid'];$ids=$delid;
$ids没有经过引号保护,直接进入SQL语句,导致SQL注入
由于通过截断,修改后面的uid,就可以导致越权操作,删除任意企业用户的企业产品。
### 漏洞证明:
拿第一处SQL注入、越权修改企业环境展示信息为例:
从代码可以看出,当查询失败时,会返回“非法操作!”:
[](https://images.seebug.org/upload/201406/101605075de1056708c72fa872b1f1edd1c23748.png)
当查询正常时,会返回“修改成功!”:
[](https://images.seebug.org/upload/201406/1016052228da846c65a8e57bd76edf1aaa7c1525.png)
[](https://images.seebug.org/upload/201406/10160555aebe788a796d6b47919ed7e9d8d23c4b.png)
这里可以看出user()的第一个字符就是r
依次遍历char的值,得到user()=root
其他几处SQL注入漏洞证明验证过程方法见漏洞:
[WooYun: PHPYUN最新版多处SQL注入及越权操作](http://www.wooyun.org/bugs/wooyun-2014-064323)
版权声明:本站所有资料均为网友推荐收集整理而来,仅供学习和研究交流使用。
工作时间:8:00-18:00
客服电话
电子邮件
admin@qq.com
扫码二维码
获取最新动态