mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2026-01-16 14:21:03 +01:00
Code Issues Projects Releases Wiki Activity

[BUG] server timeout was not considered in some circumstances

Browse Source 
Due to a copy-paste typo, the client timeout was refreshed instead
of the server's when waiting for server response. This means that
the server's timeout remained eternity.

(cherry picked from commit 9f1f24bb7fb8ebd6b43b5fee1bda0afbdbcb768e)
(cherry picked from commit df82605d3e73573ae842a1ddaf418997bef33274)
This commit is contained in:
Willy Tarreau 2008-08-11 10:35:07 +02:00
parent 4e1756086d
commit ce3f248679
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/stream_sock.c
View File

@ -174,7 +174,7 @@ int stream_sock_read(int fd) {
* have at least read something.
*/
if (b->flags & BF_PARTIAL_READ) {
if (b->flags & BF_PARTIAL_READ && tv_isset(&b->rex)) {
if (tv_add_ifset(&b->rex, &now, &b->rto))
goto out_wakeup;
out_eternity:
@ -330,13 +330,13 @@ int stream_sock_write(int fd) {
* written something.
*/
if (b->flags & BF_PARTIAL_WRITE) {
if (b->flags & BF_PARTIAL_WRITE && tv_isset(&b->wex)) {
if (tv_add_ifset(&b->wex, &now, &b->wto)) {
/* FIXME: to prevent the client from expiring read timeouts during writes,
* we refresh it. A solution would be to merge read+write timeouts into a
* unique one, although that needs some study particularly on full-duplex
* TCP connections. */
if (!(b->flags & BF_SHUTR_STATUS))
if (!(b->flags & BF_SHUTR_STATUS) && tv_isset(&b->rex))
b->rex = b->wex;
goto out_wakeup;
}