BUG: compression: properly disable compression when content-type does not match

Disabling compression based on the content-type was improperly done since the
introduction of the COMP_READY flag, sometimes resulting in truncated responses.

diff --git a/src/proto_http.c b/src/proto_http.c
index b2ae01cc1a989b18282639fd23585c108aecbfba..848e745c0d537698615ffdfac380fa76b3797c59 100644 (file)
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -2133,11 +2133,10 @@ int select_compression_response_header(struct session *s, struct buffer *res)
        return 1;
 
 fail:
-       if (s->flags & SN_COMP_READY) {
+       if (s->flags & SN_COMP_READY)
                s->comp_algo->end(&s->comp_ctx);
-               s->comp_algo = NULL;
-               s->flags &= ~SN_COMP_READY;
-       }
+       s->comp_algo = NULL;
+       s->flags &= ~SN_COMP_READY;
        return 0;
 }