Commit 46e04a73 changed svc_com_create() to avoid using
reserve ports but removed the listen() call which
could break, undocumented expectations, of legacy
server apps.
Reserve ports are not needed for listening ports, so
the new __binddynport() is used to avoid squatting
on ports that are assigned to other network services,
but it is important to say backwards compatible
with the legacy glibc code so the the listen()
call is added back.
Signed-off-by: Steve Dickson <steved@xxxxxxxxxx>
---
src/rpc_soc.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/src/rpc_soc.c b/src/rpc_soc.c
index f32a27c..cf15216 100644
--- a/src/rpc_soc.c
+++ b/src/rpc_soc.c
@@ -331,6 +331,13 @@ svc_com_create(fd, sendsize, recvsize, netid)
madefd = TRUE;
}
+ if (__binddynport(fd) == -1) {
+ if (madefd)
+ (void)close(fd);
+ (void) syslog(LOG_ERR,
+ "svc%s_create: could not bind connection: %m", netid);
+ }
+ listen(fd, SOMAXCONN);
svc = svc_tli_create(fd, nconf, NULL, sendsize, recvsize);
(void) freenetconfigent(nconf);
if (svc == NULL) {
--
2.14.3
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
