From 379f37009598b39cf46645b68486d7fb70bacdd7 Mon Sep 17 00:00:00 2001
From: Victor Hora <vhora@trustwave.com>
Date: Thu, 30 Aug 2018 18:01:53 -0400
Subject: [PATCH] Fix SecResponseBodyAccess and ctl:requestBodyAccess
 directives

---
 src/transaction.cc | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/src/transaction.cc b/src/transaction.cc
index d1f93526..4e3fdc79 100644
--- a/src/transaction.cc
+++ b/src/transaction.cc
@@ -766,14 +766,11 @@ int Transaction::processRequestBody() {
         m_variableReqbodyProcessorError.set("0", m_variableOffset);
     }
 
-#if 1
     if (m_rules->m_secRequestBodyAccess != RulesProperties::TrueConfigBoolean) {
         if (m_requestBodyAccess != RulesProperties::TrueConfigBoolean) {
 #ifndef NO_LOGS
             debug(4, "Request body processing is disabled");
 #endif
-
-            this->m_rules->evaluate(modsecurity::RequestBodyPhase, this);
             return true;
         } else {
 #ifndef NO_LOGS
@@ -786,14 +783,12 @@ int Transaction::processRequestBody() {
         if (m_requestBodyAccess == RulesProperties::FalseConfigBoolean) {
 #ifndef NO_LOGS
             debug(4, "Request body processing is enabled, but " \
-                "disable to this transaction due to ctl:requestBodyAccess " \
+                "disabled to this transaction due to ctl:requestBodyAccess " \
                 "action");
 #endif
-            this->m_rules->evaluate(modsecurity::RequestBodyPhase, this);
             return true;
         }
     }
-#endif
 
     /**
      * FIXME: This variable should be calculated on demand, it is