From: Stefan Fritsch <sf@apache.org>
Date: Tue, 25 Dec 2012 21:50:06 +0000 (+0000)
Subject: Check that AsyncRequestWorkerFactor is not negative
X-Git-Tag: 2.5.0-alpha~5949
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7e374279b5f54992de481b0a65d1165a22c3ff06;p=apache

Check that AsyncRequestWorkerFactor is not negative

PR :54254
Submitted by: Jackie Zhang <jackie qq zhang gmail com>


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

diff --git a/CHANGES b/CHANGES
index d53402ace1..27d061975a 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache 2.5.0
 
+  *) mpm_event: Check that AsyncRequestWorkerFactor is not negative. PR 54254.
+     [Jackie Zhang <jackie qq zhang gmail com>]
+
   *) httxt2dbm: Correct length computation for the 'value' stored in the DBM file
      PR 47650 [jon buckybox com]
 
diff --git a/server/mpm/event/event.c b/server/mpm/event/event.c
index 9c18af354c..23d3c0e010 100644
--- a/server/mpm/event/event.c
+++ b/server/mpm/event/event.c
@@ -3338,6 +3338,9 @@ static const char *set_worker_factor(cmd_parms * cmd, void *dummy,
     if (*endptr)
         return "error parsing value";
 
+    if (val <= 0)
+        return "AsyncRequestWorkerFactor argument must be a positive number";
+
     worker_factor = val * WORKER_FACTOR_SCALE;
     if (worker_factor == 0)
         worker_factor = 1;