Fix assert with side effects in ServerStateData::handleAdaptedHeader

setConsumerIfNotLate conditionally schedules some async calls, so the
call definitely shouldn't be removed by the precompiler.

diff --git a/src/Server.cc b/src/Server.cc
index 41d4a36c32bc8f9013c86148e282efcf19a90918..39ed4fe0f69b6e48f379735fdad362fb99b8d85b 100644 (file)
--- a/src/Server.cc
+++ b/src/Server.cc
@@ -699,7 +699,8 @@ ServerStateData::handleAdaptedHeader(HttpMsg *msg)
         // subscribe to receive adapted body
         adaptedBodySource = rep->body_pipe;
         // assume that ICAP does not auto-consume on failures
-        assert(adaptedBodySource->setConsumerIfNotLate(this));
+        const bool result = adaptedBodySource->setConsumerIfNotLate(this);
+        assert(result);
     } else {
         // no body
         if (doneWithAdaptation()) // we may still be sending virgin response