文章首发于奇安信攻防社区
链接:https://forum.butian.net/share/2905
这周看到了某公众号发的springboot+vue实现的一个后台管理系统。阅读量还挺高的,就下了一下源码翻一翻,发现里边漏洞还挺多的。尤其是SQL方面,作者虽然做了过滤但还是因为配置不当的导致SQL注入。
后边经过作者的同意,然后把本次代码审计的思路放出来和大家分享一下
mybatis-plus对绝大部分场景进行了预编译处理。但是类似动态表名、orderby这种需要拼接的场景配置不当还是会存在漏洞。本次就是记录一下在代码审计中遇到的一个奇怪的SQL注入,在对请求参数进行黑名单过滤十分严格的情况,代码逻辑不当导致SQL注入!
翻了一下pom.xml文件,发现该项目持久层使用的是Mybatis-puls框架!
我对mybatis-plus的理解就是封装了一些sql以减少代码量,其对绝大部分场景进行了预编译处理。但是类似动态表名、orderby这种需要拼接的场景配置不当还是会存在漏洞。
在翻工具类时发现该项目有对sql注入进行过滤
那就跟进去看看在哪里做了过滤,只在com.utils.Query#Query
中有4处调用
跟进com.utils.Query#Query
看看,发现该类具有两个重载的构造方法,接收的参数类型不同
JQPageInfo:封装了分页参数的实体类
Map
但其对sql注入的防御逻辑都相同,都是对sidx和order参数进行处理后,然后封装成Page对象
源码:
package com.utils;import java.util.LinkedHashMap;
import java.util.Map;
import org.apache.commons.lang3.StringUtils;
import com.baomidou.mybatisplus.plugins.Page;
/**
* 查询参数
*/
public class Query<T> extends LinkedHashMap<String, Object> {
private static final long serialVersionUID = 1L;
/**
* mybatis-plus分页参数
*/
private Page<T> page;
/**
* 当前页码
*/
private int currPage = 1;
/**
* 每页条数
*/
private int limit = 10;
public Query(JQPageInfo pageInfo) {
//分页参数
if(pageInfo.getPage()!= null){
currPage = pageInfo.getPage();
}
if(pageInfo.getLimit()!= null){
limit = pageInfo.getLimit();
}
//防止SQL注入(因为sidx、order是通过拼接SQL实现排序的,会有SQL注入风险)
String sidx = SQLFilter.sqlInject(pageInfo.getSidx());
String order = SQLFilter.sqlInject(pageInfo.getOrder());
//mybatis-plus分页
this.page = new Page<>(currPage, limit);
//排序
if(StringUtils.isNotBlank(sidx) && StringUtils.isNotBlank(order)){
this.page.setOrderByField(sidx);
this.page.setAsc("ASC".equalsIgnoreCase(order));
}
}
public Query(Map<String, Object> params){
this.putAll(params);
//分页参数
if(params.get("page") != null){
currPage = Integer.parseInt((String)params.get("page"));
}
if(params.get("limit") != null){
limit = Integer.parseInt((String)params.get("limit"));
}
this.put("offset", (currPage - 1) * limit);
this.put("page", currPage);
this.put("limit", limit);
//防止SQL注入(因为sidx、order是通过拼接SQL实现排序的,会有SQL注入风险)
String sidx = SQLFilter.sqlInject((String)params.get("sidx"));
String order = SQLFilter.sqlInject((String)params.get("order"));
this.put("sidx", sidx);
this.put("order", order);
//mybatis-plus分页
this.page = new Page<>(currPage, limit);
//排序
if(StringUtils.isNotBlank(sidx) && StringUtils.isNotBlank(order)){
this.page.setOrderByField(sidx);
this.page.setAsc("ASC".equalsIgnoreCase(order));
}
}
public Page<T> getPage() {
return page;
}
public int getCurrPage() {
return currPage;
}
public int getLimit() {
return limit;
}
}
很完美的防御、不过这里就有点奇怪了,翻了几个sql的xml文件,发现都没有使用到这个page
对象
那我们从页面中随便找一个带分页的查询请求分析一下,例如:
/jixiaokaohe/page?page=1&limit=10&sort=id
跟进到他的controller->com.controller.JixiaokaoheController#page
调用的是jixiaokaoheService.queryPage
来进行的查询,使用MPUtil.sort
来自定义了一个wrapper
继续跟进到com.service.impl.JixiaokaoheServiceImpl#queryPage
可以看到这块使用了sql过滤的com.utils.Query#Query(java.util.Map<java.lang.String,java.lang.Object>)
继续向下跟
可以看到最终sql的实现使用的是开头说的自定义的wrapper
来构建查询条件。
跟进去看看->com.utils.MPUtil#sort
发现在这块对sort
参数直接使用了拼接而且没有进行过滤
打个poc试一下吧!
page=1&limit=10&sort=id+and+updatexml(1,concat(0x7e,database()),0)#
从Mybatis-puls的简介中可以看到:Mybatis-Plus是一个Mybatis(opens new window)的增强工具,在Mybatis的基础上只做增强不做改变,为简化开发。因此mybatis中实现sql的方式在Mybatis-puls同样适用。
#{}
预处理、${}
拼接就不用多说了。直接实现sql的xml搜索${
可以发现多出使用拼接,那直接选择一个分析一下
一路向上跟进
com.service.impl.CommonServiceImpl#remindCount
最终跟到com.controller.CommonController#remindCount
,发现全程也无过滤
根据接口,构造,打个poc
users union select group_concat(SCHEMA_NAME) from information_schema.SCHEM
在对动态表名、orderby这种需要拼接的场景下,在代码实现时一定要仔细!总体来说mybatispuls框架中SQL注入漏洞挖掘相较于mybatis中更为困难,需要我们更加有耐心!
这是在无意间发现的一个功能,感觉是作者设计的一个功能。但我没太看懂作者的意图,重置密码的接口可以不进行权限认证就使用!
@IgnoreAuth
这个注解的作用是不进行权限认证,之后就是通过用户名来查询用户,然后修改用户的密码为123456
GET /springboot57n6g/users/resetPass?username=admin HTTP/1.1
Host: localhost:8081
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/122.0.0.0 Safari/537.36
Accept: application/json, text/plain, */*
sec-ch-ua: "Chromium";v="122", "Not(A:Brand";v="24", "Google Chrome";v="122"
sec-ch-ua-platform: "Windows"
sec-ch-ua-mobile: ?0
Accept-Encoding: gzip, deflate, br
Sec-Fetch-Site: same-origin
Sec-Fetch-Dest: empty
Referer: http://localhost:8081/
Sec-Fetch-Mode: cors
Accept-Language: zh-CN,zh;q=0.9
在com.controller.FileController#upload
中实现的文件上传功能,通过最后一个点来获取上传文件后缀,在结合时间生成心的文件名,但未对文件进行过滤。导致任意文件上传
在com.controller.FileController#download
中实现的文件下载功能,但是在94行文件路径是由拼接而得:静态目录/upload/filename
由于filename我们可控。并且也无过滤,导致目录穿越下载任意文件
开发注重功能的同时也应该考虑到一些安全问题。就像重置密码那个问题,作者就没有思考到安全问题。
如果你是一个长期主义者,欢迎加入我的知识星球,我们一起往前走,每日都会更新,精细化运营,微信识别二维码付费即可加入,如不满意,72 小时内可在 App 内无条件自助退款
前面有同学问我有没优惠券,这里发放100张100元的优惠券,用完今年不再发放