In article <19981109190542.BE4EA4587Eat_private>, Wietse Venema <wietseat_private> writes: > With regard to LINUX accept() returning benign error results other > than the usual suspects (EAGAIN or EINTR), Todor Tuparov asked: >> Is there a solution? >> Very simple question. > I understand that with LINUX 2.1.x the accept() call will not wake > up before the TCP three-way handshake is complete. > But for now, it is up to the programmer to deal with benign error > returns such as: ECONNREFUSED, ECONNRESET, EHOSTDOWN, EHOSTUNREACH, > ENETDOWN, ENETUNREACH, ENOTCONN, and to treat them just like EAGAIN. I'm sorry that is not the full story. In some cases Linux 2.1 TCP accept may still return network errors. This patch against 2.1.127 should fix it (and also fixes another bug). If an asynchronous error occurs now after the 3way handshake completion it will be returned on the first socket operation after accept. Index: linux/net/ipv4/tcp.c =================================================================== RCS file: /vger/u4/cvs/linux/net/ipv4/tcp.c,v retrieving revision 1.131 diff -u -u -r1.131 tcp.c --- tcp.c 1998/11/08 11:17:05 1.131 +++ tcp.c 1998/11/10 19:47:21 @@ -1630,10 +1626,12 @@ * here - you'll have to check for sk->zapped as set by the ICMP handler then. */ - error = 0; -out: release_sock(sk); + return newsk; + +out: sk->err = error; + release_sock(sk); return newsk; } Index: linux/net/ipv4/af_inet.c =================================================================== RCS file: /vger/u4/cvs/linux/net/ipv4/af_inet.c,v retrieving revision 1.80 diff -u -u -r1.80 af_inet.c --- af_inet.c 1998/11/08 11:17:03 1.80 +++ af_inet.c 1998/11/10 19:47:23 @@ -703,22 +722,20 @@ if(sk2->state == TCP_ESTABLISHED) goto do_full_success; - if(sk2->err > 0) - goto do_connect_err; err = -ECONNABORTED; if (sk2->state == TCP_CLOSE) goto do_bad_connection; do_full_success: + newsk->prot->unhash(newsk); destroy_sock(newsk); newsock->state = SS_CONNECTED; return 0; do_half_success: + newsk->prot->unhash(newsk); destroy_sock(newsk); return(0); -do_connect_err: - err = sock_error(sk2); do_bad_connection: sk2->sleep = NULL; sk2->socket = NULL; -Andi
This archive was generated by hypermail 2b30 : Fri Apr 13 2001 - 14:22:41 PDT