本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

# 前言

SQL注入漏洞作为WEB安全的最常见的漏洞之一，在java中随着预编译与各种ORM框架的使用，注入问题也越来越少。新手代码审计者往往对Java Web应用的多个框架组合而心生畏惧，不知如何下手，希望通过Mybatis框架使用不当导致的SQL注入问题为例，能够抛砖引玉给新手一些思路。

一、Mybatis的SQL注入

Mybatis的SQL语句可以基于注解的方式写在类方法上面，更多的是以xml的方式写到xml文件。Mybatis中SQL语句需要我们自己手动编写或者用generator自动生成。编写xml文件时，MyBatis支持两种参数符号，一种是#，另一种是$。比如：

Select * from news where title like ‘%#{title}%’br

使用预编译，$使用拼接SQL。

Mybatis框架下易产生SQL注入漏洞的情况主要分为以下三种：

1、模糊查询

select * from news where tile like concat(‘%’,#{title}, ‘%’)br

在这种情况下使用#程序会报错，新手程序员就把#号改成了$,这样如果java代码层面没有对用户输入的内容做处理势必会产生SQL注入漏洞。

正确写法：

select * from news where tile like concat(‘%’,#{title}, ‘%’)br

2、in 之后的多个参数

in之后多个id查询时使用# 同样会报错，

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了，我真的无语了！

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了，我真的无语了！

正确用法为使用foreach，而不是将#替换为$

id in

#{ids} 

3、order by 之后

这种场景应当在Java层面做映射，设置一个字段/表名数组，仅允许用户传入索引值。这样保证传入的字段或者表名都在白名单里面。需要注意的是在mybatis-generator自动生成的SQL语句中，order by使用的也是$，而like和in没有问题。

二、实战思路

我们使用一个开源的cms来分析，java sql注入问题适合使用反推，先搜索xml查找可能存在注入的漏洞点-->反推到DAO-->再到实现类-->再通过调用链找到前台URL，找到利用点，话不多说走起

1、idea导入项目

Idea首页 点击Get from Version Control，输入https://gitee.com/mingSoft/MCMS.git

下载完成，等待maven把项目下载完成

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

2、搜索$关键字

Ctrl+shift+F 调出Find in Path，筛选后缀xml，搜索$关键字

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

根据文件名带Dao的xml为我们需要的，以IContentDao.xml为例，双击打开，ctrl +F 搜索$,查找到16个前三个为数据库选择，跳过，

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

继续往下看到疑似order by 暂时搁置

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

继续往下看发现多个普通拼接，此点更容易利用，我们以此为例深入，只查找ids从前端哪里传入

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

3、搜索映射对象

Mybatis 的select id对应要映射的对象名，我们以getSearchCount为关键字搜索映射的对象

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

搜到了IContentDao.java,IContentDaoimpl.java和McmsAction.java，分别对应映射的对象，对象的实现类和前端controler，直接跳转到controler类

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

发现只有categoryIds与目标参数ids相似，需进一步确认，返回到IContentDao.java按照标准流继续反推

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

找到ids为getSearchCount的最后一个参数，alt+f7查看调用链

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

调转到ContentBizImpl，确认前台参数为categoryIds

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

返回到McmsAction，参数由BasicUtil.getString接收，

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

跟进BasicUtil.getString

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

继续跳到SpringUtil.getRequest()，前端未做处理，sql注入实锤

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

4、漏洞确认

项目运行起来，构造sql语句http://localhost:8080/ms-mcms/mcms/search.do?categoryId=1%27)%20%20or+updatexml(1,concat(0x7e,(SELECT+%40%40version),0x7e),1)%23 得到mysql的版本5.7.27，验证注入存在。

本以为用的MyBatis框架就万无一失了，没想到还是被黑客注入了

三、总结

以上就是mybatis的sql注入审计的基本方法，我们没有分析的几个点也有问题，新手可以尝试分析一下不同的注入点来实操一遍，相信会有更多的收获。当我们再遇到类似问题时可以考虑：

1、Mybatis框架下审计SQL注入，重点关注在三个方面like，in和order by

2、xml方式编写sql时，可以先筛选xml文件搜索$,逐个分析，要特别注意mybatis-generator的order by注入

3、Mybatis注解编写sql时方法类似

4、java层面应该做好参数检查，假定用户输入均为恶意输入，防范潜在的攻击