From: Matus Fabian <matfabia@cisco.com>
Date: Mon, 30 Sep 2024 11:52:12 +0000 (+0200)
Subject: http: http_state_wait_server_reply fix
X-Git-Tag: v25.06-rc0~254
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F12%2F41612%2F1;p=vpp.git

http: http_state_wait_server_reply fix

We enqueue 2 segments to app, first is masg (http_msg_t) and second
as much as possible of the raw data, so it must be:
max_enq - sizeof (msg)

Type: fix

Change-Id: Ib7ece7e0ad1aac99d687d49149f1bccea599b10f
Signed-off-by: Matus Fabian <matfabia@cisco.com>
---

diff --git a/src/plugins/http/http.c b/src/plugins/http/http.c
index 4f741c2e6b4..f1f59b41519 100644
--- a/src/plugins/http/http.c
+++ b/src/plugins/http/http.c
@@ -955,6 +955,7 @@ http_state_wait_server_reply (http_conn_t *hc, transport_send_params_t *sp)
    * if there is some space send also portion of body */
   as = session_get_from_handle (hc->h_pa_session_handle);
   max_enq = svm_fifo_max_enqueue (as->rx_fifo);
+  max_enq -= sizeof (msg);
   if (max_enq < hc->control_data_len)
     {
       clib_warning ("not enough room for control data in app's rx fifo");