RepoMirrors/haproxy
1
0
Fork 0
mirror of http://git.haproxy.org/git/haproxy.git/ synced 2025-04-19 13:35:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

CLEANUP: l7-retries: do not test the buffer before calling b_alloc()

Browse Source 
The return value is enough now to know if the allocation succeeded or
failed.
This commit is contained in:
Willy Tarreau 2021-03-22 16:17:37 +01:00
parent 862ad82f22
commit f499f50c8f
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/http_ana.c
View File

@ -1135,9 +1135,7 @@ int http_request_forward_body(struct stream *s, struct channel *req, int an_bit)
*/ */
DBG_TRACE_STATE("enable L7 retry, save the request", STRM_EV_STRM_ANA|STRM_EV_HTTP_ANA, s, txn); DBG_TRACE_STATE("enable L7 retry, save the request", STRM_EV_STRM_ANA|STRM_EV_HTTP_ANA, s, txn);
si->flags |= SI_FL_L7_RETRY; si->flags |= SI_FL_L7_RETRY;
if (b_is_null(&si->l7_buffer)) if (b_alloc(&si->l7_buffer) == NULL)
b_alloc(&si->l7_buffer);
if (b_is_null(&si->l7_buffer))
si->flags &= ~SI_FL_L7_RETRY; si->flags &= ~SI_FL_L7_RETRY;
else { else {
memcpy(b_orig(&si->l7_buffer), b_orig(&req->buf), b_size(&req->buf)); memcpy(b_orig(&si->l7_buffer), b_orig(&req->buf), b_size(&req->buf));