Fix #269 Guard against drivers that construct an SQLException where the 'cause' is self-referential. Hopefully the cycle is not multi-layers deep, because this check will only guard against one "loop".

10 years ago · a310df95d7
parent 953a5de33c
commit a310df95d7
1 changed files with 1 additions and 1 deletions
--- a/hikaricp-common/src/main/java/com/zaxxer/hikari/proxy/ConnectionProxy.java
+++ b/hikaricp-common/src/main/java/com/zaxxer/hikari/proxy/ConnectionProxy.java
@ -109,7 +109,7 @@ public abstract class ConnectionProxy implements IHikariConnectionProxy
            LOGGER.warn(String.format("Connection %s (%s) marked as broken because of SQLSTATE(%s), ErrorCode(%d).", delegate.toString(),
                                      parentPool.toString(), sqlState, sqle.getErrorCode()), sqle);
         }
-         else if (sqle.getNextException() instanceof SQLException) {
+         else if (sqle.getNextException() instanceof SQLException && sqle != sqle.getNextException()) {
            checkException(sqle.getNextException());
         }
      }