From: fhanik <fhanik@13f79535-47bb-0310-9956-ffa450edef68>
Date: Mon, 10 Jan 2011 16:41:43 +0000 (+0000)
Subject: https://issues.apache.org/bugzilla/show_bug.cgi?id=50333
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=c35a444ac257d8d0d39d17454aee4f6d79373f82;p=tomcat7.0

https://issues.apache.org/bugzilla/show_bug.cgi?id=50333
dont allow 0 or negative max active values


git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1057268 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java b/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
index 63bf0b725..1622c7908 100644
--- a/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
+++ b/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
@@ -402,6 +402,10 @@ public class ConnectionPool {
         } //end if
 
         //make sure the pool is properly configured
+        if (properties.getMaxActive()<1) {
+            log.warn("maxActive is smaller than 1, setting maxActive to: "+PoolProperties.DEFAULT_MAX_ACTIVE);
+            properties.setMaxActive(PoolProperties.DEFAULT_MAX_ACTIVE);
+        }
         if (properties.getMaxActive()<properties.getInitialSize()) {
             log.warn("initialSize is larger than maxActive, setting initialSize to: "+properties.getMaxActive());
             properties.setInitialSize(properties.getMaxActive());
diff --git a/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java b/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
index 70b7125d3..6fd20d406 100644
--- a/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
+++ b/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
@@ -36,6 +36,8 @@ import org.apache.juli.logging.LogFactory;
 public class PoolProperties implements PoolConfiguration {
     private static final Log log = LogFactory.getLog(PoolProperties.class);
     
+    public static final int DEFAULT_MAX_ACTIVE = 100;
+    
     protected static AtomicInteger poolCounter = new AtomicInteger(0);
     protected Properties dbProperties = new Properties();
     protected String url = null;
@@ -46,7 +48,7 @@ public class PoolProperties implements PoolConfiguration {
     protected String defaultCatalog = null;
     protected String connectionProperties;
     protected int initialSize = 10;
-    protected int maxActive = 100;
+    protected int maxActive = DEFAULT_MAX_ACTIVE;
     protected int maxIdle = maxActive;
     protected int minIdle = initialSize;
     protected int maxWait = 30000;