From ad18e2e9f143d4b66250aab2de0181e5299c5231 Mon Sep 17 00:00:00 2001 From: wenshao Date: Thu, 29 Aug 2013 16:20:33 +0800 Subject: [PATCH] =?UTF-8?q?=E5=8E=BB=E6=8E=89=E6=97=A0=E7=94=A8=E5=8F=98?= =?UTF-8?q?=E9=87=8F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/main/java/com/alibaba/druid/wall/WallProvider.java | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/main/java/com/alibaba/druid/wall/WallProvider.java b/src/main/java/com/alibaba/druid/wall/WallProvider.java index 8daaed2e1c..467a6e0836 100644 --- a/src/main/java/com/alibaba/druid/wall/WallProvider.java +++ b/src/main/java/com/alibaba/druid/wall/WallProvider.java @@ -616,7 +616,6 @@ public abstract class WallProvider { final List violations = new ArrayList(); List statementList = new ArrayList(); boolean syntaxError = false; - boolean illegal = false; try { SQLStatementParser parser = createParser(sql); parser.getLexer().setCommentHandler(WallCommentHandler.instance); @@ -631,19 +630,17 @@ public abstract class WallProvider { if (lastToken != Token.EOF) { violations.add(new IllegalSQLObjectViolation(ErrorCode.SYNTAX_ERROR, "not terminal sql, token " + lastToken, sql)); - illegal = true; } } catch (NotAllowCommentException e) { violations.add(new SyntaxErrorViolation(e, sql)); incrementCommentDeniedCount(); } catch (ParserException e) { syntaxErrrorCount.incrementAndGet(); - illegal = syntaxError = true; + syntaxError = true; if (config.isStrictSyntaxCheck()) { violations.add(new SyntaxErrorViolation(e, sql)); } } catch (Exception e) { - illegal = true; violations.add(new SyntaxErrorViolation(e, sql)); } @@ -658,7 +655,6 @@ public abstract class WallProvider { try { stmt.accept(visitor); } catch (ParserException e) { - illegal = true; violations.add(new SyntaxErrorViolation(e, sql)); } } @@ -739,7 +735,7 @@ public abstract class WallProvider { return new WallCheckResult(sqlStat); } } - + if (whiteListEnable) { WallSqlStat sqlStat = getWhiteSql(sql); if (sqlStat != null) {