From 60ceef7ea23dab008aedae6776d6afc2f0007d3b Mon Sep 17 00:00:00 2001
From: Dr Scofield <drscofield@xyzzyxyzzy.net>
Date: Tue, 20 May 2008 12:58:27 +0000
Subject: [PATCH] my last "simplification" of the the if-then-else logic in
 BaseHttpServer placed the return in the "if (request.UserAgent != null)"
 branch in the wrong place: as a result BaseHttpServer would not do anything
 if the user agent header field was present....ARGH! BAAAAAAD.

---
 OpenSim/Framework/Servers/BaseHttpServer.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/OpenSim/Framework/Servers/BaseHttpServer.cs b/OpenSim/Framework/Servers/BaseHttpServer.cs
index 51cb36eaec..a5e256be19 100644
--- a/OpenSim/Framework/Servers/BaseHttpServer.cs
+++ b/OpenSim/Framework/Servers/BaseHttpServer.cs
@@ -161,8 +161,8 @@ namespace OpenSim.Framework.Servers
                     {
                         m_log.DebugFormat("[HTTP-AGENT] Handler located for {0}", request.UserAgent);
                         HandleAgentRequest(agentHandler, request, response);
+                        return;
                     }
-                    return;
                 }
 
                 IRequestHandler requestHandler;