Mailing-List: contact commits-help@apr.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@apr.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r680511 - /apr/apr-util/trunk/misc/apr_reslist.c
Date: Mon, 28 Jul 2008 21:31:48 -0000
To: commits@apr.apache.org
From: bojan@apache.org
Message-Id: <20080728213148.B731323889FC@eris.apache.org>

Author: bojan
Date: Mon Jul 28 14:31:47 2008
New Revision: 680511

URL: http://svn.apache.org/viewvc?rev=680511&view=rev
Log:
Better enforce reslist limits.
Values cannot be negative and hmax must not be zero.

Modified:
    apr/apr-util/trunk/misc/apr_reslist.c

Modified: apr/apr-util/trunk/misc/apr_reslist.c
URL: http://svn.apache.org/viewvc/apr/apr-util/trunk/misc/apr_reslist.c?rev=680511&r1=680510&r2=680511&view=diff
==============================================================================
--- apr/apr-util/trunk/misc/apr_reslist.c (original)
+++ apr/apr-util/trunk/misc/apr_reslist.c Mon Jul 28 14:31:47 2008
@@ -245,7 +245,8 @@
 
     /* Do some sanity checks so we don't thrash around in the
      * maintenance routine later. */
-    if (min > smax || min > hmax || smax > hmax || ttl < 0) {
+    if (min < 0 || min > smax || min > hmax || smax > hmax || hmax == 0 ||
+        ttl < 0) {
         return APR_EINVAL;
     }