[代码审计]DM企业建站系统v201710 sql注入漏洞分析 | 新版v201712依旧存在sql注入

0x00 前言

本来呢,这套CMS都不想审的了。下载下来打开一看,各种debug注释,排版烂的不行。

贴几个页面看看

 

感觉像是新手练手的,没有审下去的欲望了。

但想了想,我tm就是新手啊,然后就继续看了下去。

随便点了一下seay工具自动审出来的几个关键点。发现有注入,既然有注入,就好好看看了。

phpstorm,seay源代码审计,本地demo搭起来~

看完之后,想写篇文章总结一下,标题都写好了:

DM企业建站系统v201710 存在sql注入

可能是因为代码格式不想写了,又或者其他原因

。。。

不知多少分钟之后,无意在cnvd上看到有人提交了这个系统的相关报告,而且还是两个?!

这才下定决心想捋一捋这个系统。

 

0x01 CNVD 的报告和厂商的补丁

先看看 CNVD 上的报告是怎么写的吧?

报送时间 10-23 号,挺早的呀。。。

get 到一个点,后台登录处存在sql注入~

在看看另外一份怎么写的?

哎呀,24 号,你两真有缘。

这份报告应该是系统的审了,发现了多个参数。怕是日穿了,厂商也出了补丁。

最新版是 12.1 号的。

0x03 新旧版本对比

因为我之前已经看过了旧版本的代码,也知道旧版本多参数存在注入的问题所在。

旧版本的参数过滤都是使用了这么个函数

这个函数是作者自定义的,跟进去

global.common.php

会发现用的是 htmlentities来过滤,注意函数后面第二个参数用得是 ENT_NOQUOTES。这意味着什么呢?

用函数 htmlentitiesdm 过滤的都是只将<,>编码为实体字符而已。而单双引号完美被忽视了。

那么这个过滤有何意义啊?或许作者是想着来防xss的?!

也就是说用这个函数过滤的输入,只要带到数据库前没有做过滤的都是存在注入的。。

比如CNVD第一份报告说的,后台登陆存在注入??

看到代码login.php

 

只是用 htmlentitiesdm 过滤了一下就不再过滤了,直接带进数据库,注入杠杠的。

还有第二份报告所说的多个参数注入,怕就是用了这个函数进行过滤的各个参数吧,这里就不再一一去找了。

让我们把目光放到新版本上。

打开神器beyond compare,加载新旧两个版本。

 

会发现做了很多更改。

我们单刀直入直接看看 global.common.php,看看函数 htmlentitiesdm 有没有做修改?

 

 会发现从 ENT_NOQUOTES 变成了 ENT_QUOTES。

也就是从不编码任何引号变成了编码所有引号。

也就是说这一改变把CNVD上面的两个洞都修复了。

login.php 后台登陆注入的

 

 以及其他的输入参数,都是换成了用 htmlentitdm 来过滤了。

不一一举例。

 

0x04 新版依旧存在sql注入

在对比完新旧版本之后,我就开始笑了。因为我找的几个sql注入漏洞,都没有修复。

哈哈,我也有手握0day的时候了!

1,常见的getip()函数过滤不全或者没有过滤,导致sql注入

先看旧版的 global.common.php , 很明显没有任何过滤

看看哪里用了这个函数?在前台找到两个地方

 

其中 file_formpost_concat 是用来留言的,file_order_post.php是用来下单的。‘

在我测试发现,只有留言这个功能可以用。

看代码,旧版是content和tokenhour参数都有注入的

新版的过滤掉了。

但获取ip这里是没有变的。

 

那么这里是可以拿到一个布尔类型的注入。(因为这里没有回现位)

直接 sqlmap 跑了

 

2,后台认证判断存在注入,可直接绕过登陆

 看最新版本后台认证处 admindm-yourname/config-a/common.inc2010.php

左新右旧,可以看到是没有什么改变的。

而我们仔细分析一下这段代码:

因为这里的cookie是可控的,我们可以直接进行注入,甚至不用注出密码,直接进行绕过登陆了。

我们分析代码,可以得出这样的结论。

为什么userps 也是可控的呢?因为这里存在注入啊。

通过注入,我们可以让返回的密码为我们自己输入的。

那么就可以直接构造cookie,以管理员的身份登陆进去了。

比如,我们用ps 为1 ,id 为 注入语句的cookie。

根据公式

usercookiecompare = userid-md5(userps,cookiesecet)

先构造md5部分:

那么我们的usercookie就是

7' and 1=2 union select null,null,null,null,null,1,null,null,null,null,null,null,null #-7b0bbade54ef65e2a830c034d0d14e61

把三个cookie搬上,访问一下/admindm-yourname/mod_common/index-welcome.php?lang=cn 便可直接进入后台

 

 

时间原因,就不再看其他问题了。

肯定还有其他很多问题,代码这么乱,出错的几率也增高了不少。

 

0x05 总结

总的来说,这是一套比较简单的cms,适合我们这种新手。

不知道为什么这么简单的CMS,还可以刷CNVD,CNVD真的这么简单吗?!

posted @ 2017-12-08 19:46 水泡泡 阅读(...) 评论(...) 编辑 收藏