r4757: added the ability of the clisocket level of libcli to handle async

socket connections. This was complicated by a few factors:

 - it meant moving the event context from clitransport to clisocket,
   so lots of structures changed

 - we need to asynchronously handle connection to lists of port
   numbers, not just one port number. The code internally tries each
   port in the list in turn, without ever blocking

 - the man page on how connect() is supposed to work asynchronously
   doesn't work in practice (now why doesn't this surprise me?). The
   getsockopt() for SOL_ERROR is supposed to retrieve the error, but
   in fact the next (unrelated) connect() call on the same socket also
   gets an error, though not the right error. To work around this I
   need to tear down the whole socket between each attempted port. I
   hate posix.

Note that clisocket.c still does a blocking name resolution call in
smbcli_sock_connect_byname(). That will be fixed when we add the async
NBT resolution code.

Also note that I arranged things so that every SMB connection is now
async internally, so using plain smbclient or smbtorture tests all the
async features of this new code.
(This used to be commit 468f8ebbfdbdf37c757fdc4863626aa9946a8870)

1 files changed, 0 insertions, 3 deletions

diff --git a/source4/torture/torture.c b/source4/torture/torture.c
index cc960b6ec7..d50bb9366a 100644
--- a/source4/torture/torture.c
+++ b/source4/torture/torture.c
@@ -62,8 +62,6 @@ static struct smbcli_state *open_nbt_connection(void)
 		return cli;
 	}
 
-	cli->transport->socket->timeout = 120000; /* set a really long timeout (2 minutes) */
-
 	if (!smbcli_transport_establish(cli, &calling, &called)) {
 		/*
 		 * Well, that failed, try *SMBSERVER ... 
@@ -111,7 +109,6 @@ BOOL torture_open_connection_share(struct smbcli_state **c,
 
 	(*c)->transport->options.use_oplocks = use_oplocks;
 	(*c)->transport->options.use_level2_oplocks = use_level_II_oplocks;
-	(*c)->transport->socket->timeout = 120000;
 
 	return True;
 }