From a4a583ac058dab8a2a262dcff8322ea05ce22f88 Mon Sep 17 00:00:00 2001
From: willy tarreau <willy@wtap.(none)>
Date: Sun, 18 Dec 2005 01:39:19 +0100
Subject: [PATCH] * TCP health-checks don't detect a connection refused with
 poll/epoll   because event_srv_chk_r() is called before _w() and flushes the
 socket   error. The result is that the server remains UP. The problem only  
 affects pure TCP health-checks when select() is disabled. You may   encounter
 this on SSL or SMTP proxies.

---
 haproxy.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/haproxy.c b/haproxy.c
index f6529558f4..cf9872e113 100644
--- a/haproxy.c
+++ b/haproxy.c
@@ -2849,7 +2849,8 @@ int event_srv_chk_w(int fd) {
     /* in case of TCP only, this tells us if the connection succeeded */
     if (skerr)
 	s->result = -1;
-    else {
+    else if (s->result != -1) {
+	/* we don't want to mark 'UP' a server on which we detected an error earlier */
 	if (s->proxy->options & PR_O_HTTP_CHK) {
 	    int ret;
 	    /* we want to check if this host replies to "OPTIONS / HTTP/1.0"
@@ -2886,11 +2887,11 @@ int event_srv_chk_w(int fd) {
  */
 int event_srv_chk_r(int fd) {
     char reply[64];
-    int len;
+    int len, result;
     struct task *t = fdtab[fd].owner;
     struct server *s = t->context;
 
-    s->result = len = -1;
+    result = len = -1;
 #ifndef MSG_NOSIGNAL
     {
 	int skerr;
@@ -2910,7 +2911,10 @@ int event_srv_chk_r(int fd) {
     if ((len >= sizeof("HTTP/1.0 000")) &&
 	!memcmp(reply, "HTTP/1.", 7) &&
 	(reply[9] == '2' || reply[9] == '3')) /* 2xx or 3xx */
-	s->result = 1;
+	result = 1;
+
+    if (s->result != -1)
+	s->result = result;
 
     FD_CLR(fd, StaticReadEvent);
     task_wakeup(&rq, t);