tree dfc0ac036859d68441cd92df1f8006e01665816a
parent bd55e3167b7b7f179062a86c6219defe12e5ad47
author Willy Tarreau <w@1wt.eu> 1289469364 +0100
committer Willy Tarreau <w@1wt.eu> 1289469920 +0100
encoding latin1

[BUG] accept: don't close twice upon error

The stream_sock's accept() used to close the FD upon error, but this
was also sometimes performed by the frontend's accept() called via the
session's accept(). Those interlaced calls were also responsible for the
spaghetti-looking error unrolling code in session.c and stream_sock.c.

Now the frontend must not close the FD anymore, the session is responsible
for that. It also takes care of just closing the FD or also removing from
the FD lists, depending on its state. The socket-level accept() does not
have to care about that anymore.