[代码审计]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真的这么简单吗？！