lib/async_req: remove useless "reentrant" support from async_connect_send()

Now that open_socket_out*() doesn't do the strange reentrant calls,
we can remove support for this in async_connect_send(), so that we'll
never get any new users of this.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>

diff --git a/lib/async_req/async_sock.c b/lib/async_req/async_sock.c
index d0cb06b0638258030e2e3791acb79e263467fd5b..e436d00ac798cd2ff96348ba658da3925a0d5540 100644 (file)
--- a/lib/async_req/async_sock.c
+++ b/lib/async_req/async_sock.c
@@ -131,10 +131,6 @@ struct tevent_req *async_connect_send(
         * The only errno indicating that an initial connect is still
         * in flight is EINPROGRESS.
         *
-        * We get EALREADY when someone calls us a second time for a
-        * given fd and the connect is still in flight (and returned
-        * EINPROGRESS the first time).
-        *
         * This allows callers like open_socket_out_send() to reuse
         * fds and call us with an fd for which the connect is still
         * in flight. The proper thing to do for callers would be
@@ -142,7 +138,7 @@ struct tevent_req *async_connect_send(
         * socket.
         */
 
-       if (errno != EINPROGRESS && errno != EALREADY) {
+       if (errno != EINPROGRESS) {
                tevent_req_error(req, errno);
                return tevent_req_post(req, ev);
        }