Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
From:   "NeilBrown" <neilb@suse.de>
To:     Steve Dickson <steved@redhat.com>
Cc:     linux-nfs@vger.kernel.org
Subject: [PATCH nfs-utils] mount: don't bind a socket needlessly.
Date:   Tue, 02 Nov 2021 14:08:48 +1100
Message-id: <163582252847.13683.7467712657489228784@noble.neil.brown.name>
Precedence: bulk


When clnt_ping() calls get_socket(), get_socket() will create a socket,
call bind() to choose an unused local port, and then connect to the
given address.

The "bind()" call is unnecessary and problematic.
It is unnecessary as the "connect()" call will bind the socket as
required.
It is problematic as it requires a completely unused port number, rather
than just a port number which isn't currently in use for connecting to
the given remote address.
If all local ports (net.ipv4.ip_local_port_range) are in use, the bind()
will fail.  However the connect() call will only fail if all those port
are in use for connecting to the same address.

So remove the unnecessary bind() call.

Signed-off-by: NeilBrown <neilb@suse.de>
---
 utils/mount/network.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/utils/mount/network.c b/utils/mount/network.c
index e803dbbe5a2c..35261171a615 100644
--- a/utils/mount/network.c
+++ b/utils/mount/network.c
@@ -429,10 +429,6 @@ static int get_socket(struct sockaddr_in *saddr, unsigne=
d int p_prot,
 	if (resvp) {
 		if (bindresvport(so, &laddr) < 0)
 			goto err_bindresvport;
-	} else {
-		cc =3D bind(so, SAFE_SOCKADDR(&laddr), namelen);
-		if (cc < 0)
-			goto err_bind;
 	}
 	if (type =3D=3D SOCK_STREAM || (conn && type =3D=3D SOCK_DGRAM)) {
 		cc =3D connect_to(so, SAFE_SOCKADDR(saddr), namelen,
--=20
2.33.1