ORDER BY字段必须白名单校验,禁止参数化拼接;需严格校验排序字段(如"id","name")、方向("ASC"/"DESC")及多字段拆分,反引号别名须额外过滤,校验须前置到Controller层且全覆盖所有导出入口。
Spring Boot 导出 Excel 时若支持用户指定排序字段(如 ?sort_field=created_at&sort_order=desc),ORDER BY 后的内容无法用 ? 或 :param 占位符处理——JDBC 和所有主流 ORM 都不支持对排序字段做参数绑定。直接拼接 "ORDER BY " + request.getParameter("sort_field") 等同于把 SQL 解析权交给攻击者。
常见错误是试图用正则过滤或转义单引号,但攻击者可绕过:sort_field=1; DROP TABLE users--、sort_field=(SELECT password FROM admins)、sort_field=`status` ASC, (SELECT 1) -- 均可成功执行。
private static final Set<string> ALLOWED_SORT_FIELDS = Set.of("id", "name", "email", "created_at", "amount");</string>
.toLowerCase() 后比对ASC/DESC)也需单独白名单校验,禁止透传任意字符串,例如 Set.of("ASC", "DESC")
sort_field=name,created_at),需拆分后逐个校验,任一不匹配即拒用 @Select 注解或 XML 写原生 SQL 时,${} 是危险的(非参数化,直接替换),而 #{} 对 ORDER BY 无效(会加引号导致语法错误)。所以不能靠 MyBatis 的占位符“蒙混过关”。
正确做法是:在 Service 层完成白名单校验后,再拼接安全的 SQL 片段。例如:
String sortField = request.getParameter("sort_field");String sortOrder = request.getParameter("sort_order");if (!ALLOWED_SORT_FIELDS.contains(sortField.toLowerCase()) || !ALLOWED_SORT_ORDERS.contains(sortOrder.toUpperCase())) { throw new IllegalArgumentException("Invalid sort parameter");}String safeOrderBy = String.format("%s %s", sortField, sortOrder);// 再传入 MyBatis 的 ${},此时已无风险
ORDER BY ${safeOrderBy},但前提是 safeOrderBy 已经是校验后的纯字母+空格+ASC/DESC@Param("sortField") String sortField 并传给 ${} —— 校验必须在进入 DAO 层前完成wrapper.orderBy(true, true, sortField),因为 sortField 未校验;应先校验,再调用 wrapper.orderByAsc("id").orderByDesc("created_at") 等固定链式调用Sort.by("user_input") 看似安全,实则危险:它底层仍会将字符串拼进 SQL 的 ORDER BY 子句,且不做白名单检查。传入 new Sort(Sort.Direction.ASC, "name; DROP TABLE users--") 可触发注入。
JPA 的 Pageable 构造函数同样不校验字段名,PageRequest.of(0, 10, Sort.by("status")) 中的 "status" 必须来自可信源。
request.getParameter("sort") 塞进 Sort.by()
Sort 实例,例如:Map.of("created_at", Sort.by(Sort.Direction.DESC, "created_at"))
Sort.by(Sort.Order.asc("name"), Sort.Order.desc("created_at")),而非拼字符串Sort.Order 构造器接受字段名,但不校验;所以映射逻辑必须前置有些导出需求允许用户自定义列别名,例如 SELECT name AS `Full Name`, email AS `Contact Email` FROM users。反引号内的内容若来自用户输入,可能逃逸并闭合引号,插入恶意 SQL。
比如用户传 alias=`status` ASC, (SELECT 1)--,拼进去变成 SELECT status AS `status` ASC, (SELECT 1)--` FROM users,破坏语法甚至执行子查询。
Map.of("name", "姓名", "email", "邮箱"))full_name),并拒绝任何反引号、双引号、单引号字符真正的难点不在写校验逻辑,而在确认所有导出入口——包括后台管理页、API 文档测试框、Swagger UI、Postman 收藏夹里的旧请求——都经过同一套白名单校验。漏掉一个,就等于整条防线失效。