tcp: Actually enforce MAX_CONNS limit

and, given that the connection table is indexed by socket number,
we also need to increase MAX_CONNS now as the ICMP implementation
needs 2^17 sockets, that will be opened before TCP connections are
accepted.

This needs to be changed later: the connection table should be
indexed by a translated number -- we're wasting 2^17 table entries
otherwise. Move initialisation of TCP listening sockets as last
per-protocol initialisation, this will make it easier.

Signed-off-by: Stefano Brivio <sbrivio@redhat.com>

1 files changed, 1 insertions, 1 deletions

diff --git a/passt.c b/passt.c
index 12b10dc..466cae8 100644
--- a/passt.c
+++ b/passt.c
@@ -787,7 +787,7 @@ int main(int argc, char **argv)
 
 	fd_unix = sock_unix();
 
-	if (icmp_sock_init(&c) || tcp_sock_init(&c) || udp_sock_init(&c))
+	if (icmp_sock_init(&c) || udp_sock_init(&c) || tcp_sock_init(&c))
 		exit(EXIT_FAILURE);
 
 	if (c.v6)