From 9d5f54daad88ddc67b75a6a34682e5bf9335f1aa Mon Sep 17 00:00:00 2001 From: William Lallemand Date: Tue, 14 Nov 2017 14:39:22 +0100 Subject: [PATCH] BUG/MEDIUM: cache: use msg->sov to forward header Use msg->sov to forward headers instead of msg->eoh. It can causes some problem because eoh does not contains the last \r\n, and the filter does not support to send the headers partially. Cc: Christopher Faulet --- src/cache.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/cache.c b/src/cache.c index 02bb0e048..26d9b12af 100644 --- a/src/cache.c +++ b/src/cache.c @@ -137,7 +137,7 @@ cache_store_http_headers(struct stream *s, struct filter *filter, struct http_ms if (!(msg->chn->flags & CF_ISRESP) || !st) return 1; - st->hdrs_len = msg->eoh; + st->hdrs_len = msg->sov; return 1; } @@ -394,12 +394,12 @@ enum act_return http_action_store_cache(struct act_rule *rule, struct proxy *px, if (!(txn->flags & TX_CACHEABLE)) goto out; - if ((msg->eoh + msg->body_len) > (global.tune.bufsize - global.tune.maxrewrite)) + if ((msg->sov + msg->body_len) > (global.tune.bufsize - global.tune.maxrewrite)) goto out; shctx_lock(shctx); - first = shctx_row_reserve_hot(shctx, sizeof(struct cache_entry) + msg->eoh + msg->body_len); + first = shctx_row_reserve_hot(shctx, sizeof(struct cache_entry) + msg->sov + msg->body_len); if (!first) { shctx_unlock(shctx); goto out; @@ -417,7 +417,7 @@ enum act_return http_action_store_cache(struct act_rule *rule, struct proxy *px, /* does not need to be locked because it's in the "hot" list, * copy the headers */ - if (shctx_row_data_append(shctx, first, (unsigned char *)s->res.buf->p, msg->eoh) < 0) + if (shctx_row_data_append(shctx, first, (unsigned char *)s->res.buf->p, msg->sov) < 0) goto out; /* register the buffer in the filter ctx for filling it with data*/ -- 2.47.3