Vtiger CRM 几处SQL注入漏洞分析，测试工程师可借鉴

本文由云+社区发表

0x00 前言

干白盒审计有小半年了，大部分是业务上的代码，逻辑的复杂度和功能模块结构都比较简单，干久了收获也就一般，有机会接触一个成熟的产品（vtiger crm）进行白盒审计，从审计的技术难度上来说，都比公司内的那些业务复杂得多，而真正要提高自己技术水平，更应该看的也是这些代码。

vtiger crm是一个客户关系管理系统。

0x01 分析整体结构

https://www.vtiger.com/open-source-crm/download-open-source/代码下载下来，本地搭建。使用phpstorm进行审计。

1. 主目录下的vtigerversion.php可以查看当前版本。

1. 整体代码目录

   

其中主要得功能实现就在modules目录当中，也是我们重点审计的地方。libraries目录是使用到的第三方的一些东西，includes目录是路由加载，封装系统函数的地方。

整个系统代码量确实很多，真要审计完估计没有十天半个月是不行的，看了一个礼拜，只发现几个问题。

0x02 modules/calender/actions/feed.php sql注入分析

一个成熟的产品，审计的难点就在于各种类，对象的封装和继承，a调用b，b调用c，c调用d......

vtiger_basicajax_action 这个对象，是modules下vtiger目录里的，而vtiger这个也是核心的module.

回到feed.php,直接定位有漏洞的代码，103行后。

我图中标的，也正是注入点的位置。

$fieldname参数由逗号分割成数组，如果分成后的数组值为2则进入逻辑，然后参数进入sql语句形成注入。虽然整个系统采用了pdo的查询方式，但是如果有sql语句存在直接拼接的话，还是有注入的风险。

这里payload不能使用逗号，可以采用 (select user())a join的方法绕过。

往下走的话，sql注入漏洞更是多不胜数。也没有再看的必要了。

0x03 /modules/documents/models/listview.php sql注入

直接看漏洞代码

可以看到sortorder参数又是直接拼接。此处是order by后的注入，只能用基于时间的盲注。

直接上sqlmap吧，但是sqlmap的payload会使用>，尖括号因为xss防御，已经被过滤所以需要使用绕过脚本。 --tamper greatest 绕过。

poc:

index.php?module=documents&parent=&page=1&view=list&viewname=22&orderby=filename&sortorder=and/**/sleep(5)&app=marketing&search_params=[]&tag_params=[]&nolistcache=0&list_headers=[%22notes_title%22,%22filename%22,%22modifiedtime%22,%22assigned_user_id%22,%22filelocationtype%22,%22filestatus%22]&tag=

0x04 写在最后

由于时间原因，只看了前几个模块，还有好多地方没有看。

漏洞都很简单，真正花费时间的是走通逻辑，验证漏洞，不停地跳转查看函数调用，和各种类对象的继承。这也是白盒审计的头疼之处，要忍着性子看开发跳来跳去，没准哪个地方就跳错了。有点难受，还没找到getshell的地方。

此文已由作者授权腾讯云+社区发布

---