From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:37442 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751546AbeDKPIa (ORCPT ); Wed, 11 Apr 2018 11:08:30 -0400 From: Steve Dickson To: Libtirpc-devel Mailing List Cc: Linux NFS Mailing list Subject: [PATCH] clnt_create: Restore using reserve ports for client connections Date: Wed, 11 Apr 2018 11:08:03 -0400 Message-Id: <20180411150803.26680-1-steved@redhat.com> Sender: linux-nfs-owner@vger.kernel.org List-ID: Commit 46e04a73 changed clnt_create(3) not to use reserve ports when binding a connection to a server. For certain legacy apps, like some of the NIS commands, the client has to used reserve port to be able to communicate with the server. So the use of reserve ports in clnt_create(3) has been restored. Signed-off-by: Steve Dickson --- src/clnt_generic.c | 3 +-- src/rpc_soc.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/clnt_generic.c b/src/clnt_generic.c index e5a314f..774292b 100644 --- a/src/clnt_generic.c +++ b/src/clnt_generic.c @@ -341,8 +341,7 @@ clnt_tli_create(int fd, const struct netconfig *nconf, servtype = nconf->nc_semantics; if (!__rpc_fd2sockinfo(fd, &si)) goto err; - if (__binddynport(fd) == -1) - goto err; + bindresvport(fd, NULL); } else { if (!__rpc_fd2sockinfo(fd, &si)) goto err; diff --git a/src/rpc_soc.c b/src/rpc_soc.c index af6c482..f32a27c 100644 --- a/src/rpc_soc.c +++ b/src/rpc_soc.c @@ -147,8 +147,7 @@ clnt_com_create(raddr, prog, vers, sockp, sendsz, recvsz, tp, flags) bindaddr.maxlen = bindaddr.len = sizeof (struct sockaddr_in); bindaddr.buf = raddr; - if (__binddynport(fd) == -1) - goto err; + bindresvport(fd, NULL); cl = clnt_tli_create(fd, nconf, &bindaddr, prog, vers, sendsz, recvsz); if (cl) { -- 2.14.3