From 4322f95717ea09671d8b8f08bdc2fe3fa8e77bb4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E9=AB=98=E9=93=81?= <shaojin.wensj@alibaba-inc.com>
Date: Sun, 18 Apr 2021 16:51:20 +0800
Subject: [PATCH] bug fixed for connection valid check, fix #4227

---
 .../java/com/alibaba/druid/pool/DruidAbstractDataSource.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/java/com/alibaba/druid/pool/DruidAbstractDataSource.java b/src/main/java/com/alibaba/druid/pool/DruidAbstractDataSource.java
index 74622ad7f6..f5b5ef40c6 100644
--- a/src/main/java/com/alibaba/druid/pool/DruidAbstractDataSource.java
+++ b/src/main/java/com/alibaba/druid/pool/DruidAbstractDataSource.java
@@ -1388,7 +1388,7 @@ public abstract class DruidAbstractDataSource extends WrapperAdapter implements
         }
 
         if (validConnectionChecker != null) {
-            boolean result = true;
+            boolean result;
             Exception error = null;
             try {
                 result = validConnectionChecker.isValidConnection(conn, validationQuery, validationQueryTimeout);
@@ -1406,6 +1406,7 @@ public abstract class DruidAbstractDataSource extends WrapperAdapter implements
             } catch (SQLException ex) {
                 throw ex;
             } catch (Exception ex) {
+                result = false;
                 error = ex;
             }