|
* Y" _: U8 F* D: [
1.1.Sql 注入攻击原理SQL 注入漏洞可以说是在企业运营中会遇到的最具破坏性的漏洞之一,它也是目前被利用得最多的漏洞要学会如何防御 SQL 注入,首先我们要学习它的原理针对 SQL 注入的攻击行为可描述为通过在用户可控参数中注入 SQL 语法,破坏原有 SQL 结构,达到编写程序时意料之外结果的攻击行为。
, _! z5 K$ Z+ R. W/ A3 ? 其成因可以归结为以下两个原因叠加造成的:程序编写者在处理应用程序和数据库交互时,使用字符串拼接的方式构造 SQL 语句未对用户可控参数进行足够的过滤便将参数内容拼接进入到 SQL 语句中注入攻击的本质,是把用户输入的数据当做代码执行。 $ o8 M+ o5 U8 _; o
这里有两个关键条件:用户能够控制输入原本程序要执行的代码,拼接了用户输入的数据1.2.Sql 审计方法手动找的话,可以直接找到 sqlmapper.xml 文件或者直接搜索 select、update、delete、insert “String sql=”等关键词,定位 SQL xml 配置文件。 . v. V1 m% a5 m O" G
如果 sql 语句中有出现 $ 进行参数拼接,则存在 SQL 注入风险当找到某个变量关键词有 SQL 注入风险时,可以再根据调用链找到该存在注入风险的业务逻辑代码,查看参数来源是否安全、是否有配置 SQL 危险参数过滤的过滤器,最终确认是否存在 SQL 注入。 & S- H) q E" K
以下给出可能造成 sql 注入攻击的关键字,审计时可根据实际情况进项查找常见 SQL 语句关键词【一一帮助安全学习,所有资源一一】①网络安全学习路线②20 份渗透测试电子书③安全攻防 357 页笔记④50 份安全攻防面试指南⑤安全红队渗透工具包⑥网络安全必备书籍⑦100 个漏洞实战案例⑧安全大厂内部教程 ! j- g" L# c: P9 W$ m
1.3Sql 注入漏洞危害1 、 攻击者可以做到业务运营的所有数据被攻击对当前数据库用户拥有的所有表数据进行增、删、改、查等操作若当前数据库用户拥有 file_priv 权限,攻击者可通过植入木马的方式进一步控制 DB 所在服务器
7 L2 R+ l2 L4 ~4 E6 ` 若当前数据库用户为高权限用户,攻击者甚至可以直接执行服务器命令从而通过该漏洞直接威胁整个内网系统2、可能对业务造成的影响① 用户信息被篡改② 攻击者偷取代码和用户数据恶意获取线上代码被非法篡改,并造成为恶意攻击者输送流量或其他利益的影响
8 ]+ F/ W% m5 ]0 t y 1.4Sql 注入漏洞代码示例Java 代码动态构建 SQLStatement stmt = null;
`% S" U' u6 u7 ~- Q% ?
& Q8 w! o$ y9 g2 t/ ~8 ^: c$ g- d+ @0 K) w5 ?
ResultSet rs = null;
6 u+ u5 T9 x! [$ v8 C# a3 E6 t4 M+ @, k
+ i9 L8 |! W' z8 X& r try{+ a2 b) k, p) T4 I, t+ @, s8 ^
3 k6 U! s4 ^3 k4 u7 \
& O% V. _& |5 W+ f- M$ T* v$ h String userName = ctx.getAuthenticatedUserName();
) ~4 i" c$ E. K7 A& { //this is a constantString sqlString = "SELECT * FROM t_item WHERE owner=" + userName + " AND itemName=" 8 q; k" ?3 `! h% r) f7 e z
+ request.getParameter("itemName") + "";" ?' l# ?% W' W% r2 ^& i* c: o
$ u" R+ r( d% q
" {- E/ P8 z6 D stmt = connection.createStatement();! U/ z: v! w8 ?- G3 g. v0 K
# Y4 m3 M7 y+ a* ^" Z- i
4 Z3 Y4 O) z7 w$ z- E* Y/ a* c rs = stmt.executeQuery(sqlString);
/ J% G/ b6 |; x! H: r* g t) ^* N* Q2 p& T! K
4 T/ T$ l! O- d+ ~+ S5 u
' @9 _3 w# ] B4 L // ... result set handling
1 P a7 q. `# e! c" B* a4 N$ X ?
2 q, I7 U) d, M' V: r% E7 |! m% z3 P9 q2 n. }
}
1 @ }; Y0 [6 L
% o% m. A8 |+ ^8 b0 i# A6 K% H% F1 Q# n
catch (SQLException se){
6 e) j; {/ c% {( o
$ k- f, n# r7 u& Q/ I3 W8 P
4 w( r; W+ Q K) [ // ... logging and error handling
+ J# A4 l9 u$ q
# `: w. Y! X% S9 {$ f4 N0 t' l- R6 `1 J1 s8 _' b) c' ]
}
3 g6 y7 n" o d+ T4 h8 n6 S/ p/ f6 w$ ~/ p
复制代码
5 a7 Y/ U* g9 C) G/ p 这里将查询字符串常量与用户输入进行拼接来动态构建 SQL 查询命令仅当 itemName 不包含单引号时,这条查询语句的行为才会是正确的如果一个攻击者以用户名 wiley 发起一个请求,并使用以下条目名称参数进行查询:。 - Y( k5 U/ ^' V P0 S$ x
name OR a = a8 w3 X/ V6 T( u) H5 r! f
) z. {1 W% F( }/ h4 O: K 复制代码那么这个查询将变成:SELECT * FROM t_item WHERE owner = wileyAND itemname = nameORa=a;
) c2 k5 f8 g" x/ G- V! u1 Z5 T
复制代码此处,额外的 OR a=a条件导致整个 WHERE 子句的值总为真。
1 T7 e$ c2 M8 ^* F8 F7 Q 那么,这个查询便等价于如下非常简单的查询:SELECT * FROM t_item; Y! ~1 c- \/ R+ O) l1 `
4 c; q" n0 F7 M# ^$ Y
复制代码这个简化的查询使得攻击者能够绕过原有的条件限制:这个查询会返回 items 表中所有储存的条目,而不管它们的所有者是谁,而原本应该只返回属于当前已认证用户的条目。 4 `# |2 f6 N3 P( z. L
在存储过程中动态构建 SQLJava 代码:CallableStatement = null
8 M6 A" |9 I. _% D5 G
: O$ Y) p9 b% i: M
# d E* G( D1 t" K) w ResultSet results = null;$ I4 @5 y! e, X# o/ ^" q
+ b# F$ q. v5 r1 o- ?! }. U) |' z7 x, {/ d$ v0 F F$ \% {
try
1 k" e* U1 ?. S
7 p- x' I, o& {+ {
' n: f5 E) a+ G {
7 ^1 W& n, O# B% O
0 X0 |7 U' D# U0 e! b6 T( n( O% w2 f3 b. h y
String userName = ctx.getAuthenticatedUserName(); # s, |7 a. ?3 u1 P( ^2 r9 I
//this is a constant
- d: f' d \, l F
, B& M4 [; a; A) }( G
1 a7 }; Z k6 q String itemName = request.getParameter("itemName");- {3 z0 s5 v# W _ v9 t# v
) K7 T) b& R% l. }# \
% x9 m; U2 v* l" L4 w+ K- s8 { cs = connection.prepareCall( + o6 N) E/ o7 ]- v5 z% B2 ]
"{call sp_queryItem(?,?)}");. M5 L- u9 K8 R' L, k3 z8 D
7 H4 Y) @. E5 _
! h8 m) ~! O" l( P; W; c, } cs.setString(1, userName);
$ }7 x: H1 X+ b! V7 V% l! Q- Y9 n5 v% m; I2 Y5 j1 D. o8 c [- [
& k+ ^. ?8 N( b cs.setString(2, itemName);
% v/ S( Q, _0 l" [" {4 n
+ `. p: |9 ?+ w
1 G- @) c- W! h) r results = cs.executeQuery();
& q- }1 W. E1 V q1 D' n n; [; R1 H( v4 k
5 B! \7 s o3 j' r 4 B! \5 p& @/ L2 {% l
// ... result set handling
' ]0 @: }# ~0 }' A5 q5 G
7 w9 O/ o, S1 \/ C7 _ r8 Y @9 e6 H$ R" e; k' S
}
3 V4 Q$ t% f% ?8 J4 ~" p0 ^) p% A4 Y/ e! W0 [' i
, A `9 a% g- S1 a' b3 X catch (SQLException se)
$ t# n! I" {" Q) N- ]3 {) Z8 Q
& r9 N" \2 @; k# W9 U. G# n: G3 W& o3 {8 B
{6 R1 p4 U& L( e* Q
+ f7 @4 N, o1 @/ I8 _/ X
9 `& e3 @1 U r/ H+ N* @0 u+ t8 x8 l // ... logging and error handling/ P. }. c2 E$ T8 d/ p
+ j* O% }/ M1 q+ H7 c0 r- d
! \/ z/ X0 L- K Z" \ }
' F" S, b9 ^& M) @7 n7 @2 W4 R, P5 I) x& A) q
; o+ F& k/ |, Y3 h SQL Server存储过程:% \3 d8 c" F. H+ m0 t
0 ~9 I9 E) f/ @9 m @) h
! w9 M! z! i& ?& S9 p+ Z+ u CREATE PROCEDURE sp_queryItem! x8 D/ m) i4 A4 P6 a+ W
9 c* {# {5 p% x8 l! ^% w
( W1 e) G" I3 o) C" y @userName varchar(
$ @; p; W! a; }; V, q! [# o6 o6 ] 50),0 R [% z; S6 h. m
) W0 O: p/ K4 N3 d
8 U7 _ H, z) Q0 `5 N1 P @itemName varchar(50)" R. T: \( j' T0 m( H
# Q4 ~8 K# ~1 S* q* p: m+ W1 U' ^3 ^
1 n" N# }! d/ q4 M- B; J/ G AS
! U! U, T, q/ A0 N
' I) T, S: v4 p U/ J3 Q7 t: ~" ]1 @) Z5 Z! M7 n
BEGIN
( O) q, T5 S2 K8 b, W" T' {- k) @% ?' F, D* P; ], {8 `$ e- J1 J
; x5 ]. m0 }- v7 ^1 M& `( e0 }5 n1 w9 B
DECLARE @sql nvarchar(500);
2 a& K! P" {$ S3 m" L$ p' P3 z- x4 S% e
4 T7 |' B( O T9 o7 L SET @sql = SELECT * FROM t_item
3 X+ \( s/ [2 e, R2 J/ F' Z3 H& W
" Y! l) A* s5 y) ~0 c& k WHERE owner =
- {. X( C* L F: w9 X, b + @userName +
7 z# l" U" d$ b3 ^
9 k k; d; s" H. @4 r( P' G( m: K; [! K" V# s% `! \, c+ ?' k; e
AND itemName = + @itemName + ;
* i9 O- h" \" J- O3 }" F% s- W9 l$ e ]
4 W1 C9 }7 ]( a2 E I EXEC(@sql);' T' T6 x" ?2 ?6 i
: t" p' ?+ Z" s, D# I
* n* P6 P7 h5 j% Z, n; W END
8 J! [ s- m" H- ~! S, H# Y' C( Y$ k$ v7 c* p0 P
w2 {% w v* x0 z0 y
GO
) V. L+ \$ W1 }1 b8 S3 O' r. _
$ W% n4 H+ ?7 c) `* u 复制代码在存储过程中,通过拼接参数值来构建查询字符串,和在应用程序代码中拼接参数一样,同样是有 SQL 注入风险的。 + c" x/ R; Q2 [1 K. [3 ~
Hibernate 动态构建 SQL/HQL原生 SQL 查询:String userName = ctx.getAuthenticatedUserName(); //this is a constant
3 S6 A+ r/ |1 W, i 5 k0 C) Z; w$ V; j( r3 I% r
& l3 |2 G& Y2 Z, ?( z. P
, l$ I: G" v3 b1 V1 h$ D String itemName = request.getParameter("itemName");
1 \3 z* l, V# W' `
6 L1 N5 N/ ^# R* K: ?: o4 R ] K. Y7 q
Query sqlQuery = session.createSQLQuery("select * from t_item where owner = " . P7 c# K+ w9 N' \! W4 x
+ userName + " and itemName = " + itemName + "");, i* P0 M: v; E4 J
0 ~' I) n: T! g8 T
; a# {' G" E6 U0 Y& z' i List rs = (List) sqlQuery.list();
3 `: P' s" f$ c% S( [1 j7 m& H; {( Z
) Q) E5 M6 n) \, f4 X 复制代码 ! c; J) f; [* H+ D/ y
HQL 查询:String userName = ctx.getAuthenticatedUserName(); //this is a constant! p4 c3 i* i( S6 ~8 d4 b2 L) l
2 A7 I# e; E/ D( L0 O
/ |) ^8 D5 f3 g
String itemName = request.getParameter( , M3 Z1 I" P- h
"itemName");
- y v# c6 P1 x! S
: K4 q7 [. C. M1 h. h* c
3 _$ _0 X r5 v4 e. v Query hqlQuery = session.createQuery("from Item as item where item.owner = " + userName +
5 a9 C% s$ A% Q5 V " and item.itemName = " + itemName + "");
! D( {7 [) V2 M$ f8 u$ J3 `6 U7 f% ~6 W L* {! G5 j
+ [0 u" ]( F' F, f7 J4 o8 d- n. b
List hrs = (List) hqlQuery.list();0 h5 @# B- g& X$ h! V$ J4 p
' b: P* O0 G3 R$ W5 ~ 复制代码即使是使用 Hibernate,如果在动态构建 SQL/HQL 查询时包含了不可信输入,同样也会面临 SQL/HQL 注入的问题。 : b1 V! d- S" X& j5 Y" Y+ P
HQL 代码中,session.createQuery 使用 HQL 语句将查询到的数据存到到 list 集合中,需要时在拿出来使用而参数中 itemName 是通过 request.getParameter 直接获取。
+ e; w6 j/ ~) D5 P5 | 攻击者若在此处写入恶意语句,程序将恶意语句查询出来的数据存放在 list 集合中,再通过某处调用成功将数据显示在前台Mybatis 注入分析Mybatis 框架下易产生 SQL 注入漏洞的情况主要分为以下三种: + Z+ r% }* P# x" P2 M7 D# T
1)模糊查询 like例如对人员姓名检索进行模糊查询,如果考虑安全编码规范问题,其对应的 SQL 语句如下:Select * fromuserwherenamelike%#{name}%复制代码但由于这样写程序会报错,研发人员将 SQL 查询语句修改如下: 9 Z+ r' m- l- L0 Z9 j
Select * fromuserwherenamelike%${name}%复制代码在这种情况下我们发现程序不再报错,但是此时产生了 SQL 语句拼接问题,如果 java 代码层面没有对用户输入的内容做处理势必会产生 SQL 注入漏洞。 4 U+ j/ k+ |& s+ l2 W! |6 u
2)in 之后的参数例如对人员姓名进行同条件多值检索的时候,如当用户输入 001,002,003...时,如果考虑安全编码规范问题,其对应的 SQL 语句如下:Select * fromnamewhere
7 l# ^2 M5 W. U3 m \ idin (#{id})复制代码但由于这样写程序会报错,研发人员将 SQL 查询语句修改如下:Select * fromnamewhereidin (${id})
2 e. v: U( F& H1 f( h9 g b- |: W6 I! O
复制代码修改 SQL 语句之后,程序停止报错,但是却引入了 SQL 语句拼接的问题,如果没有对用户输入的内容做过滤,势必会产生 SQL 注入漏洞。
) c7 g' K& C2 b3 k. B% ` G$ f 3)order by 之后(重点和区分点)当根据姓名、id 序号等信息用户进行排序的时候,如果考虑安全编码规范问题,其对应的 SQL 语句如下:Select * fromuserwherename = : \6 M, Y+ b. Y( e7 o
qihooorderby#{id} desc复制代码但由于发布时间 id 不是用户输入的参数,无法使用预编译研发人员将 SQL 查询语句修改如下:Select * fromuserwherename = / b; W% j; R- B/ S3 j
qihooorderby ${id} desc复制代码修改之后,程序未通过预编译,但是产生了 SQL 语句拼接问题,极有可能引发 SQL 注入漏洞1.5.实战案例-OFCMS SQL 注入漏洞分析本文中使用 ofcms 进行 SQL 注入漏洞讲解,此 CMS 算是对新手学习代码审计比较友好的 CMS。
# E) b4 M% w$ Y! x4 H7 r 上述为安装成功页面,如何安装 CMS 本章不在赘述。后台页面:http://localhost:8080/ofcms-admin/admin/index.html & c$ j6 H, C$ A& i. P
漏洞点:ofcms-admin/src/main/java/com/ofsoft/cms/admin/controller/system/SystemGeneratrController.java8 K! s7 _" Q' f$ ^+ ]" p# ]& Y
6 R2 l/ R" E. A% S) p* V( l& _# ?: S0 `
create方法
- Q) q! Z. {6 H1 D3 V; b9 q& {& U) _: X! }9 t$ j% S. u+ V, n
- i! }: j: N& Z$ S+ p4 o | f( ^2 f) j9 C U
' t! y5 l4 h1 ^! R5 A" x
, q+ ?' j) e w2 ]# l9 t* j
& x) v" E+ w i4 d i# @5 F /**1 Q! i; H, T3 s8 u# W
' o; }8 y( t0 V% t9 \& k
3 I: @ U: q J! @; s * 创建表
0 z1 B) s' x3 t8 Y4 c( d
' E+ I2 i4 r' L g
% ~! [" ? _4 o \! N! t% W) K$ w */publicvoidcreate() {# c% b9 f0 J T- I) c$ {$ K8 H- H" U
# I! z% R; w0 [9 g' l/ X& B- X6 F) B* L1 j
try {
9 i& `9 ?1 t8 R
; ~; O. Z7 ]% h1 f; x* S+ B/ j3 b1 _, @( M: b+ t+ r" D" \) b4 b2 A4 }
String sql = getPara("sql");0 ~ D X& m/ b( k1 _
! S& {2 ?8 R4 a& T! O( E# X8 A
$ p' D, R* U4 i+ g _2 h9 U Db.update(sql);
" C) q( n$ p7 Z+ F' l) F$ L" `% E$ u2 Y! Q
+ P6 G6 O/ i# W, ~4 T/ O U
rendSuccessJson();
5 g+ _. Y+ |; q M# k5 r9 J
* ^, @% j- R1 b, W
4 W( R5 o3 q* x9 W+ {4 b' o+ N }
: j9 B+ r* e* m5 n2 p catch (Exception e) {
7 }( [3 k- H/ j, V% e0 ?6 q
* \% c" O' u, I: A6 N6 q
* P6 o; q8 C9 L( k3 t. r/ I+ } e.printStackTrace();
% @8 c& e" W# D2 X- z% @: F* y |
7 \1 q g2 G3 ~$ i. h2 w6 v rendFailedJson(ErrorCode.get("9999"), e.getMessage());
2 ]/ Y- D6 ^+ O3 S4 x$ U! G# t2 |* J8 _3 T
S5 i# ^9 p: K+ ?$ {9 x }
. i; \9 Q. V: ^1 U+ c4 l( I# s* C- G
+ T6 m+ z2 e) O) {
4 `! f1 y' M, B( x }
3 X/ c* {& a! c, x( ^9 v$ L
3 u6 V# F" H" z
6 n$ H9 W+ S2 B 复制代码上述代码中使用 getpara 获取 sql 的参数值,并 update,跟进一下 getpara 和 update 方法跳转至jfinal-3.2.jar/com/jfinal/core/controller.class。 # {% ?+ R: ~, A4 ^* h* _" _) B% C
public String getPara(String name) {* M4 }' g4 d) f! j& T
9 Y4 U1 j9 x; G
3 I2 R& Q) n/ N3 n/ O returnthis.request.getParameter(name);
7 T. M$ c( N! f4 V8 U8 `
$ m4 ?' l- {/ p( h8 Q
V4 J7 H6 c& S, `3 F }' p( x! Z/ Z( p: A; b
& d: H" Z& }2 f4 Y& ~ 复制代码上述代码无特殊用意,就是获取参数值,继续跟进 Db.update 方法。
+ H5 A: |4 m$ w& W 跳转至jfinal-3.2.jar/com/jfinal/plugin/activerecord/Db.classpublicstaticintupdate(String sql){4 m2 t$ t. u. v3 X- |% i
- ?9 v9 t6 b2 j3 d+ K
3 o+ p, P: ]# C6 D' Z# { return MAIN.update(sql);
# k' V8 q# R3 W4 d | j
) T" R6 [: }6 B/ e/ d8 {
) C1 \; O: q( V7 Y }0 S% N4 h# o0 S% z6 }* }8 Y: @) Z
0 k% z- }- A2 |* \* B
, B0 V+ O: v" M/ c
复制代码发现调用 MAIN.update , 继续跟进跳转至jfinal-3.2.jar/com/jfinal/plugin/activerecord/DbPro.classpublicintupdate。 5 w; d5 t- }, Q! x8 j4 E( z
(String sql){
( i" Y8 F3 E( S. e4 W3 U
/ e+ j% [- b6 y- G0 A- J+ J1 R
+ i. f& ?" a. ] a; B9 z1 _" i returnthis.update(sql, DbKit.NULL_PARA_ARRAY);
0 C; V+ ^( ~; ~$ ~4 ~: D) t! Q* w1 O* l) r' w3 L* e
" \1 ]# ^2 N3 Y% n
}
6 Q& L6 }3 d+ \8 k5 D4 z
% g( h* N3 b. R: i. i o$ I7 ?! J 复制代码继续跟进到最后,发现华点public int update(String sql, Object... paras) {
3 y" u9 B/ r% U9 B; z; y k* H& i$ E- ~1 b" T& O
& H" B+ ^ |6 e3 b. u% B+ n" ^- f
Connection conn = 。
! N- e& x$ y. o1 u8 m/ z5 O null;" m6 V) l3 n& A* O( |
- L# g* ?" P6 F1 E- [
; j0 L% {+ f7 E& A: h( R$ u int var4;
6 I% }6 S+ k# Y5 S; q7 h6 }3 a" g0 C0 E% U& ]8 P1 t
- T1 a7 N& `% c* p; @6 s. T try {
2 Z+ H5 X7 [6 Q, V8 R: }6 g+ z* f% r& u
2 r. B% r- u; g% s3 b0 k
conn = this.config.getConnection();//连接* b# l( w( f7 b6 E4 l9 B$ G
7 B3 v+ n! }+ N4 j& x
8 ? V) B6 u* x& ^
var4 = this.update(this.config, conn, sql, paras);
* C- W1 \* Q% c0 C& k //调用update更新
$ ]& K: ^, p& Q6 q% n. e
5 z6 l. k; S, t5 [: O# p, |& z+ u" J F" @4 F0 A* T* z
} catch (Exception var8) {# \. l K2 X9 L" X" W; _; r
9 S2 H; m# Q: ~6 d2 k
4 b( e1 M4 w' n3 i throw new ActiveRecordException(var8);& G R# }) J) h) S! J# s' e
* {7 G4 ]* f3 X. A+ @! i' c) N6 j9 v2 b- e" m3 o# R5 R
} finally {
4 A8 `1 \" W& ?, u. s, f1 j0 A5 ~( Y
- n F% A& z7 f
this / O! Y" F$ ^/ D2 _
.config.close(conn);4 l" b- H. d d d$ B: D; G/ Q8 x
" m6 J2 b* h1 f/ T; c& h P r! F
& A/ e; R1 O1 o6 Q }
: E* e) ?9 e6 e+ e' T- e- z! v- [/ p4 K6 ^7 Y) v0 W
) P2 j, h5 \, j" t
return var4;( @0 Z0 F5 K7 M' I- |
/ _7 y# S& w2 o5 q4 X
( W0 J/ d* O& |$ X
}- a; x4 t1 _( w3 s
/ [' v2 r) q& ~$ W3 p 复制代码重点:Object...Object 是所有类的基类,而 Object... 是不确定方法参数情况下的一种多态表现形式(可以传递多个参数)。
( r4 ~; V5 S4 ]! G 再继续跟进 update ,同文件代码intupdate(Config config, Connection conn, String sql, Object... paras)throws SQLException " H4 Q) l6 R# M! g
{3 A$ c# g6 { X8 E- s
0 g. y0 q5 K! x' w% X, n, a
* `3 }* R4 P! y1 t4 I% X PreparedStatement pst = conn.prepareStatement(sql);. t. y7 t6 Z7 A) S( h
/ l( p" \9 C4 G0 U- f" `0 q' f1 a7 I
config.dialect.fillStatement(pst, paras);2 s6 G1 z7 `4 J+ C. y# r4 B8 ]
( o, T0 K7 {: b! t9 t% K5 u) b! S4 U1 x# A
6 o% @5 E9 |: J, `1 A6 `* Z int result = pst.executeUpdate();
& P6 n4 [8 w2 B* N6 l7 d, I1 H/ k$ T4 \7 T& k
/ F# _( D6 S+ @' q {) C
DbKit.close(pst);
0 L/ w8 k+ u. H7 ~- s# P5 U$ S
& T. d! C8 V/ V2 [: s* k& o) \+ F: x M! r; d0 D
return result;
, C4 Z, b, A. Q3 I8 ]! [4 q; U) X O7 p6 A1 O+ g+ }; B
* I5 b {3 C6 g+ e, V
}
$ t. U. k; N1 `/ @. N7 L* R {% Y" e1 [- Q
复制代码上述代码执行 SQL 语句,并返回结果至此,整个功能流程结束,在我们跟进的过程中,代码中无任何过滤语句,获取参数值,调用 update 方法更新,更新成功后返回结果。
1 {0 a3 l2 @ p' S6 {2 E; N 漏洞验证
, O. ]! x& S1 k5 j4 ~3 A/ p 漏洞点打上断点,网页中输入 poc 进行验证update of_cms_topic set topic_url=updatexml(1,concat(0x7e,(user())),0) where topic_id = 2 K# i: S3 w9 ]) k9 D2 W0 t; _4 ]
1复制代码   
: p& x# s- a1 [9 }( R* n& l& v8 ` 根据如上截图可看出我们传入的 SQL 语句是被完整的接收,并未做任何过滤直接带入数据库执行,所以此处直接写入漏洞代码爆出当前数据库账户为 root上述为 sqlmap 工具跑出来的注入点1.6漏洞修复方法。
# I1 M1 d) p" f5 a6 g9 U# ?+ l 添加全局过滤器,过滤特殊字符
8 F6 t1 E( k$ }% {) R2 u: p SQLFilter.java 中: " r0 z" b# H0 `+ A
PreparedStatement 参数化如果使用参数化查询,则在 SQL 语句中使用占位符表示需在运行时确定的参数值参数化查询使得 SQL 查询的语义逻辑被预先定义,而实际的查询参数值则等到程序运行时再确定。 2 }0 }- J# x0 \3 Z ?& @
参数化查询使得数据库能够区分 SQL 语句中语义逻辑和数据参数,以确保用户输入无法改变预期的 SQL 查询语义逻辑在 Java 中,可以使用 java.sql.PreparedStatement 来对数据库发起参数化查询。 + {8 V+ x* H/ R: R$ n
在这个正确示例中,如果一个攻击者将 itemName 输入为 name OR a = a,这个参数化查询将免受攻击,而是会查找一个 itemName 匹配 name OR a = a 这个字符串的条目PreparedStatement stmt =
- G! F& o8 M0 ? null
3 R+ B4 I- P$ K) Z( }$ ~4 b4 J1 U5 z5 y2 u$ Z
- _ K- [7 n2 [3 E t% @
ResultSet rs = nulltry' |3 p. c; d' w3 S% ^& g* g
# O, v6 O' m) v* j
: U! D9 q3 K/ s1 @
{
% `9 v* W7 J# l5 z8 C* Q8 o
9 M- P9 u0 c6 g f4 h& D! [' X% C2 S6 R: l
String userName = ctx.getAuthenticatedUserName(); //this is a constant
6 w! l' Y- c, U) e5 O String itemName = request.getParameter("itemName");/ R& ]4 p5 d; q6 K1 z7 y
/ `6 @( e/ t2 e; m4 X. r$ b
& h& G( C& i: U- o9 P
// ...Ensure that the length of userName and itemName is legitimate
$ A/ F# U+ v4 W- l( [ // ...String sqlString = "SELECT * FROM t_item WHERE owner=? AND itemName=?";9 H" g0 m3 p* F5 A0 c( I" E8 Z
: ?- Y8 N6 G- @! p$ j- l2 }4 c# x" c4 s( {. n
stmt = connection.prepareStatement(sqlString);+ R" m; T; g+ k
' X2 Z8 }! I6 F4 @& t
/ Y1 w) P+ A$ n5 u5 x stmt.setString( 7 d, g7 o0 Q( T4 L8 O8 Q
1, userName);
6 ^: M2 C5 H+ ^# m! |, H0 M6 O; C* W0 L) ] q
* k) r% A+ H* X8 O: P9 F stmt.setString(2, itemName);
( y2 p+ q& S, U5 Q4 ]: M! G
+ v/ e* ?$ p; z6 ~6 g4 M# k2 t# P8 ]: s; {! c+ `2 H3 m
rs = stmt.executeQuery();
/ p4 G! k* _7 m$ h9 P! E/ P
) b5 ?# D( C3 |" q6 d" j
4 Q4 W5 y9 `6 N% y6 r7 M // ... result set handling
* ~9 x' [( Z: }6 ~! }
! H1 j, h- ]) O/ d4 @6 s0 L6 a& s; l0 }4 h) X: U
: \3 N* G4 [9 _0 z. Z7 Q2 p
}( O: m! |8 }5 `( v$ d8 C+ }# ?
% T& b( _2 g1 [ I7 B, T) Y
7 a8 }2 o7 [) J: _6 Y3 r catch (SQLException se)! M2 q6 B/ P8 W# u
. b" c; ~- W4 ?) C4 N7 u
7 M: ]0 l s/ W {
6 _7 T0 m- B/ L& F: b5 M# f6 q% j; s' R |
0 m. w9 A( v' ^8 k c+ G* S f+ S // ... logging and error handling
* c7 `, b% ?- u, a& U, B1 p* H$ p( [) }
* @1 T" U) V4 Z- ?5 c% ` }
- i( |1 E# I D d+ b" [# r) b0 S2 R7 o$ q/ r6 s
复制代码存储过程参数化这个存储过程使用参数化查询,而未包含不安全的动态 SQL 构建。 - L2 x9 |" y' _5 t5 D5 \1 |% B! ^
数据库编译此存储过程时,会生成一个 SELECT 查询的执行计划,只允许原始的 SQL 语义被执行任何参数值,即使是被注入的 SQL 语句也不会被执行,因为它们不是执行计划的一部分CallableStatement = , @" _0 `+ h3 ?) q
null1 ?& @- {9 i+ ], b5 `' w: c& P7 F
% k9 R: C y9 D% ^& I& D, v
* B( k2 j4 W" |: f
ResultSet results = null;
/ P( b- ?# e# R" C& A6 N6 O; ?. Q) T/ n
! Y8 @4 H6 e; v try
7 x5 N* v: Y/ Y" @
$ F2 W2 Q; ]+ |5 | r/ S. q, F) B8 n: m8 H1 Q. A7 N; u1 B# c+ \
{
# x4 X8 P* [4 K8 j) d5 i/ G
) r$ f! T$ f. l+ ?; O$ ^% T) n: c% A* C1 |8 G- b
String userName = ctx.getAuthenticatedUserName(); //this is a constant
/ ^ g, Z; S5 ~8 a* W String itemName = request.getParameter("itemName");
2 D9 V% A3 A4 M& a3 A
/ p# [6 n& _; s8 h$ i; Y+ \
+ o2 n, E' T/ Q" Q // ... Ensure that the length of userName and itemName is legitimate 6 S, R) T" ~$ h! q( z. y
// ...& [1 u3 L( U1 o- z
% n( Q/ v' H0 y. [
- h1 ]8 B' y% W5 M cs = connection.prepareCall("{call sp_queryItem(?,?)}");
4 ^; Y2 I5 Z9 p
* E- R% Y3 ]0 B& p! n6 T# t
. w, D$ s9 h$ H+ g! p cs.setString(1, userName);
# Q4 w7 s: C1 ]( f- P2 c" n3 h" c; b' X$ o/ N6 L4 ~
# }2 L3 C. k+ w: c) p" d
cs.setString(
! O2 J/ M- u( e, E5 S0 ]2 q 2, itemName);
! s% C# m9 t0 b6 d0 q1 W# g- c) T# T; ^; d% o
G: s# x0 G& P' M% m1 h3 F results = cs.executeQuery();
. N* c \0 ^4 X/ n: p9 B
" W$ F3 ^" z5 f4 Q3 U* q% s. I3 K3 A! l( F4 C- `3 {. v: \ e
// ... result set handling
0 L+ k3 p* N6 O9 @2 R, G3 }2 k
( h5 j. D6 o, S- D3 r
5 ~ R/ R" B B }
, Z( b9 x V5 q2 B: A
9 T @% X; y: g& X) i; g1 i* h- h. z$ ]7 Y% a
catch (SQLException se)
1 Y W# L: V' G0 G d
& H5 V& i- h+ d" J2 X; E4 y
& M) u6 z7 ]' Y' z" B {: T- v& r% {# _- Z; s9 C* P4 @
r3 Q7 ?3 h" S- ~5 i8 v! x1 ~) K( n& h; M' i7 B( `, E
# a! L; ^7 [- P: }! ?
// ... logging and error handling) {6 W( o# a# m4 ^
* Z5 \' O% ~! z7 r4 m6 s, Q7 k
. r# W; r1 k) t+ {) x2 o }* P% Q) p% S$ t4 f- |2 ?0 X
* D, `% h8 {- ]8 r% ~( u0 H. I 复制代码Hibernate 参数化查询Hibernate 支持 SQL/HQL 参数化查询为了防止 SQL 注入以及改善性能,以上这些示例使用了参数化绑定 的方式来设置查询参数。
& X, ^$ O L4 e String userName = ctx.getAuthenticatedUserName(); //this is a constant
: ^( b0 K: D/ |' W' ]3 H) ]) a, C$ e2 ^' G0 _
" o& v% z& z& ~, j" a# _ String itemName = request.getParameter(
+ N( H4 A+ J, u9 w; Q8 x3 J "itemName");
1 v4 A: a, r7 T. O6 ?! r( q8 B( z
$ _$ ~9 y/ T R+ v, _' I( ?: m' J* K& y w
Query hqlQuery = session.createQuery("from Item as item where item.owner = ? and item.itemName = ?"
$ X* J1 x# {- N; M- |( U1 s' a );. G' _1 q" `! q1 e# A# n
, Q4 H) `* |+ [8 x4 G Q# a8 i5 N, \
* N4 f2 Z0 }5 ?' V, L$ D+ {7 A( C' N
hqlQuery.setString(1, userName);) f2 @ M7 N3 m d
5 N! M( Z4 V! L, _. n7 F0 J2 \# j1 a) a }# J1 ?! }8 G
hqlQuery.setString(2, itemName);% n: I, n2 w& ]5 K
) o4 s! d5 M8 Z1 L7 o! |
# l% X- W: h& Q8 V1 F1 Y9 p, } List rs = (List) hqlQuery.
- V ? V1 d0 S& [. e& b& ~: M list();
; q- |* g- ~# w! L/ p) y) X: @1 m' _* L8 f! k0 m" U4 B- {
复制代码HQL 基于名称的参数化查询String userName = ctx.getAuthenticatedUserName(); //this is a constant
+ a& Z u: s1 k" E( w. r
( Y8 A; [$ @5 ?& P7 w+ W/ X) c* S' S. J0 a, R& W
String itemName = request.getParameter( 3 D g4 n7 G, }. p' S- o
"itemName");7 Q* d! N9 e/ j3 ?
/ G8 W" @/ i: I. e. `8 |: P
# x' u; A( b4 b, j; ` Query hqlQuery = session.createQuery("from Item as item where item.owner = wner and item.itemName = :itemName"
! m+ }5 o2 g0 C0 F( ?+ R2 [4 ~& q );
9 @2 D; x& W3 T$ j3 b2 a& W6 H2 v* T+ ^' {/ L
3 M [% X( t* A; I! j& K h
hqlQuery.setString("owner", userName);) _# a, n% X$ L( `# C
' |: D, ~7 L7 ~& E' z( _
1 L7 W Z7 E* f& }6 h8 s hqlQuery.setString("itemName", itemName);; f. q+ V3 _+ L7 Q; z0 ]
; ]) d* `+ K' f* C8 @& Y0 x( D" z; U- @4 w6 u4 |6 a% z4 ?
List rs = ( 9 ]: ?& s3 v* F7 Q" y( i
List) hqlQuery.list();7 z8 ]/ J, |0 T5 y' @
8 A4 ?0 N( b* P" U2 {3 W( Z1 r 复制代码原生参数化查询String userName = ctx.getAuthenticatedUserName(); //this is a constant 2 T' s# c& n7 Z# n
" }, ]% @8 J$ U. O" Y7 o6 V6 E$ ~0 P# o2 k! G/ k# S
; w) E7 [5 ^* K$ z/ ^' _
String itemName = request.getParameter("itemName");: a# s7 `$ q/ _9 X1 o
1 _7 C1 a6 _+ `- _
* p, \- @& l9 _: {# @+ v Query sqlQuery = session.createSQLQuery("select * from t_item where owner = ? and itemName = ?" " @, J+ N5 B% a8 @) i5 Q* U" `0 G
);
9 x$ k* u" W7 f( Z
/ D1 [9 x0 [ r3 q9 b4 v( U \& |* c6 B# @) R8 h7 P
sqlQuery.setString(0, owner);! i$ N/ ^2 `7 s _: I; P% r& F* s
/ {- P {# P0 [4 ~- L1 x
7 z0 p: @' a; |1 ~& D1 A* k- f: z sqlQuery.setString(1, itemName);; i" ]9 z( o3 i. O9 Z8 F
; R6 ^) n# V: j( @
2 Z& B B; m. E5 g1 Z List rs = (List) sqlQuery. ' w6 `' c6 y, l# B
list();
$ x7 L1 C0 Z+ c4 i4 G, n2 }
/ E+ h) {! H- q& d% ` ?$ G" t 复制代码MyBatis 框架的修复方案尽量使用 #描述参数,如果一定要使用 $,则需要自己过滤用户输入模糊查询 like SQL 注入修复建议按照新闻标题对新闻进行模糊查询,可将 SQL 查询语句设计如下: 4 X4 L* m B+ _, L# @" g
select * from news wherenamelikeconcat(‘%’,#{name }, ‘%’)复制代码采用预编译机制,避免了 SQL 语句拼接的问题,从根源上防止了 SQL 注入漏洞的产生。 ; m- e6 I& H; Y0 b& T4 d
in 之后的参数 SQL 注入修复建议在对新闻进行同条件多值查询的时候,可使用 Mybatis 自带循环指令解决 SQL 语句动态拼接的问题:select * from news whereidin
# g& p3 L6 ]" f# g "ids" item="item"open="("separator=","close=")">#{item} 复制代码order by SQL 注入修复建议在 Java 层面做映射预编译机制只能处理查询参数,其他地方还需要研发人员根据具体情况来解决。 - u" V& q& k1 H
如前面提到的排序情景:Select * from news where title =‘淘宝’ orderby#{time} asc,复制代码这里 time 不是查询参数,无法使用预编译机制,只能这样拼接: & f8 N# \1 z! Q5 `9 O
Select * from news where title =‘淘宝’ orderby ${time} asc复制代码针对这种情况研发人员可以在 java 层面做映射来进行解决如当存在发布时间 time 和点击量 click 两种排序选择时,我们可以限制用户只能输入 1 和 2。
+ O7 U6 {8 K* s8 [4 p' o$ h 当用户输入 1 时,我们在代码层面将其映射为 time,当用户输入 2 时,将其映射为 click而当用户输入 1 和 2 之外的其他内容时,我们可以将其转换为默认排序选择 time(或者 click)。
2 N9 P+ O& k* E, D& l1 }' w# s0 A
- j" B6 ]% k' }6 a1 i2 F, z& d
6 ~& t5 j' c. s' ~, D& x
; W- K( m; E7 U6 o |