Seems to me that if we have a worker which is usable, we

should return that. Otherwise we have no worker (worker
is NULL) and we should return that or else the if clause
takes care of setting worker correctly.

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

diff --git a/modules/proxy/proxy_balancer.c b/modules/proxy/proxy_balancer.c
index 1c307c753f4f989e974ac3fb1eb4ee1e321b1aac..8984e02b9c73816d5e8e5210c21bbeecbd51104c 100644 (file)
--- a/modules/proxy/proxy_balancer.c
+++ b/modules/proxy/proxy_balancer.c
@@ -194,8 +194,6 @@ static proxy_worker *find_session_route(proxy_balancer *balancer,
             if (worker && !PROXY_WORKER_IS_USABLE(worker))
                 worker = NULL;
         }
-        else
-            worker = NULL;
         return worker;
     }
     else