Performance: short-circuit reply_body_max_size ACL tests

In the common default case there are no reply body limits configured.
There is no need to construct ACL checklists for testing. This saves
one allocation and several locking/unlocking cycles per request.

diff --git a/src/HttpReply.cc b/src/HttpReply.cc
index ac088509bfe9fb835f494b4f5ed1f160ba0b5733..bfa84dbf2268ff2a5beb39854d77742323aa0c4f 100644 (file)
--- a/src/HttpReply.cc
+++ b/src/HttpReply.cc
@@ -589,6 +589,10 @@ HttpReply::calcMaxBodySize(HttpRequest& request)
         return;
     bodySizeMax = -1;
 
+    // short-circuit ACL testing if there are none configured
+    if (!Config.ReplyBodySize)
+        return;
+
     ACLFilledChecklist ch(NULL, &request, NULL);
 #if FOLLOW_X_FORWARDED_FOR
     if (Config.onoff.acl_uses_indirect_client)