ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

http: http_state_wait_server_reply fix

Browse Source 
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>
This commit is contained in:
Matus Fabian
2024-09-30 13:52:12 +02:00
parent 5a4c7a9ce4
commit 38e94c3461
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/plugins/http/http.c
View File

@ -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 */ * if there is some space send also portion of body */
as = session_get_from_handle (hc->h_pa_session_handle); as = session_get_from_handle (hc->h_pa_session_handle);
max_enq = svm_fifo_max_enqueue (as->rx_fifo); max_enq = svm_fifo_max_enqueue (as->rx_fifo);
max_enq -= sizeof (msg);
if (max_enq < hc->control_data_len) if (max_enq < hc->control_data_len)
{ {
clib_warning ("not enough room for control data in app's rx fifo"); clib_warning ("not enough room for control data in app's rx fifo");