From patchwork Fri Feb 1 17:21:31 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chuck Lever X-Patchwork-Id: 217539 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from aserp1040.oracle.com (aserp1040.oracle.com [141.146.126.69]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "aserp1040.oracle.com", Issuer "VeriSign Class 3 International Server CA - G3" (not verified)) by ozlabs.org (Postfix) with ESMTPS id 6BCA32C0095 for ; Sat, 2 Feb 2013 04:21:39 +1100 (EST) Received: from acsinet22.oracle.com (acsinet22.oracle.com [141.146.126.238]) by aserp1040.oracle.com (Sentrion-MTA-4.3.1/Sentrion-MTA-4.3.1) with ESMTP id r11HLaLK026087 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Fri, 1 Feb 2013 17:21:36 GMT Received: from oss.oracle.com (oss-external.oracle.com [137.254.96.51]) by acsinet22.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id r11HLZPi020438 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 1 Feb 2013 17:21:36 GMT Received: from localhost ([127.0.0.1] helo=oss.oracle.com) by oss.oracle.com with esmtp (Exim 4.63) (envelope-from ) id 1U1KIx-0006hG-Px; Fri, 01 Feb 2013 09:21:35 -0800 Received: from acsinet22.oracle.com ([141.146.126.238]) by oss.oracle.com with esmtp (Exim 4.63) (envelope-from ) id 1U1KIw-0006h3-8V for fedfs-utils-devel@oss.oracle.com; Fri, 01 Feb 2013 09:21:34 -0800 Received: from aserp1020.oracle.com (aserp1020.oracle.com [141.146.126.67]) by acsinet22.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id r11HLXnq020358 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Fri, 1 Feb 2013 17:21:34 GMT Received: from mail-ia0-f175.google.com (mail-ia0-f175.google.com [209.85.210.175]) by aserp1020.oracle.com (Sentrion-MTA-4.2.2/Sentrion-MTA-4.2.2) with ESMTP id r11HLXAA003249 (version=TLSv1/SSLv3 cipher=RC4-SHA bits=128 verify=OK) for ; Fri, 1 Feb 2013 17:21:33 GMT Received: by mail-ia0-f175.google.com with SMTP id r4so5680573iaj.34 for ; Fri, 01 Feb 2013 09:21:32 -0800 (PST) X-Received: by 10.42.153.70 with SMTP id l6mr10114408icw.50.1359739292774; Fri, 01 Feb 2013 09:21:32 -0800 (PST) Received: from seurat.1015granger.net (adsl-99-26-161-222.dsl.sfldmi.sbcglobal.net. [99.26.161.222]) by mx.google.com with ESMTPS id ww6sm2996786igb.2.2013.02.01.09.21.31 (version=TLSv1 cipher=RC4-SHA bits=128/128); Fri, 01 Feb 2013 09:21:32 -0800 (PST) From: Chuck Lever To: fedfs-utils-devel@oss.oracle.com Date: Fri, 01 Feb 2013 12:21:31 -0500 Message-ID: <20130201172130.63192.73056.stgit@seurat.1015granger.net> In-Reply-To: <20130201171714.63192.68358.stgit@seurat.1015granger.net> References: <20130201171714.63192.68358.stgit@seurat.1015granger.net> User-Agent: StGIT/0.14.3 MIME-Version: 1.0 X-Flow-Control-Info: class=Default reputation=ipRepBelow100 ip=209.85.210.175 ct-class=R5 ct-vol1=0 ct-vol2=8 ct-vol3=8 ct-risk=50 ct-spam1=80 ct-spam2=5 ct-bulk=5 rcpts=1 size=4018 X-MM-CT-Classification: not spam X-MM-CT-RefID: str=0001.0A090204.510BF99D.0077,ss=1,re=0.000,fgs=0 Subject: [fedfs-utils] [PATCH 12/13] Clean up use of 2049 as default NFS server port X-BeenThere: fedfs-utils-devel@oss.oracle.com X-Mailman-Version: 2.1.9 Precedence: list Reply-To: fedfs-utils Developers List-Id: fedfs-utils Developers List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: fedfs-utils-devel-bounces@oss.oracle.com Errors-To: fedfs-utils-devel-bounces@oss.oracle.com X-Source-IP: acsinet22.oracle.com [141.146.126.238] Clean up: Replace local definition of the NFS_PORT port macro with the correct public header definition. Introduced by commit 4f681570 "libjunction: Add support for nfs-basic junctions", Thu Dec 22 15:40:01 2011. Clean up: Avoid using "zero" for the fileserver port when creating new NFS FSL records. Signed-off-by: Chuck Lever --- src/libjunction/nfs.c | 6 ++---- src/libnsdb/administrator.c | 4 +++- src/nfsref/lookup.c | 7 ++++++- src/nsdbc/nsdb-create-fsl.c | 3 ++- src/nsdbc/nsdb-resolve-fsn.c | 3 ++- 5 files changed, 15 insertions(+), 8 deletions(-) diff --git a/src/libjunction/nfs.c b/src/libjunction/nfs.c index 42c1015..3447bb5 100644 --- a/src/libjunction/nfs.c +++ b/src/libjunction/nfs.c @@ -77,16 +77,14 @@ #include #include +#include + #include "fedfs.h" #include "nsdb.h" #include "junction.h" #include "junction-internal.h" #include "xlog.h" -#ifndef NFS_PORT -#define NFS_PORT (2049) -#endif - /** * Tag name of NFS location element of a junction XML document */ diff --git a/src/libnsdb/administrator.c b/src/libnsdb/administrator.c index 0565caa..4fd93ec 100644 --- a/src/libnsdb/administrator.c +++ b/src/libnsdb/administrator.c @@ -41,6 +41,8 @@ #include #include +#include + #include "nsdb.h" #include "junction.h" #include "nsdb-internal.h" @@ -627,7 +629,7 @@ nsdb_nfsfsl_to_uri(const struct fedfs_nfs_fsl *nfsfsl, UriUriA *uri) nsdb_assign_textrange(&uri->scheme, "nfs"); nsdb_assign_textrange(&uri->hostText, nfsfsl->fn_fslhost); - if (nfsfsl->fn_fslport != 2049 && nfsfsl->fn_fslport != 0) { + if (nfsfsl->fn_fslport != NFS_PORT && nfsfsl->fn_fslport != 0) { char portbuf[8]; sprintf(portbuf, "%u", nfsfsl->fn_fslport); nsdb_assign_textrange(&uri->portText, portbuf); diff --git a/src/nfsref/lookup.c b/src/nfsref/lookup.c index 6c8764d..1271e3d 100644 --- a/src/nfsref/lookup.c +++ b/src/nfsref/lookup.c @@ -28,6 +28,8 @@ #include #include +#include + #include "fedfs.h" #include "junction.h" #include "nsdb.h" @@ -194,9 +196,12 @@ static void nfsref_lookup_display_fedfs_nfs_fsl(struct fedfs_nfs_fsl *nfsl) { char *rootpath; + unsigned port = nfsl->fn_fslport; printf("\tFileserver name:\t\t%s\n", nfsl->fn_fslhost); - printf("\tFileserver port:\t\t%u\n", nfsl->fn_fslport); + if (port == 0) + port = NFS_PORT; + printf("\tFileserver port:\t\t%u\n", port); if (nsdb_path_array_to_posix(nfsl->fn_nfspath, &rootpath) == FEDFS_OK) { printf("\tExport path:\t\t\t%s\n", rootpath); diff --git a/src/nsdbc/nsdb-create-fsl.c b/src/nsdbc/nsdb-create-fsl.c index 0e15e15..8af3177 100644 --- a/src/nsdbc/nsdb-create-fsl.c +++ b/src/nsdbc/nsdb-create-fsl.c @@ -40,6 +40,7 @@ #include #include +#include #include #include "fedfs.h" @@ -135,7 +136,7 @@ main(int argc, char **argv) nsdb_env(&nsdbname, &nsdbport, &binddn, &nce); - serverport = 0; + serverport = NFS_PORT; while ((arg = getopt_long(argc, argv, nsdb_create_fsl_opts, nsdb_create_fsl_longopts, NULL)) != -1) { switch (arg) { diff --git a/src/nsdbc/nsdb-resolve-fsn.c b/src/nsdbc/nsdb-resolve-fsn.c index 5a004fb..38ec28d 100644 --- a/src/nsdbc/nsdb-resolve-fsn.c +++ b/src/nsdbc/nsdb-resolve-fsn.c @@ -40,6 +40,7 @@ #include #include +#include #include #include "fedfs.h" @@ -115,7 +116,7 @@ nsdb_resolve_fsn_display_nfs_fsl(struct fedfs_nfs_fsl *nfsl) if (status != FEDFS_OK) return; - if (nfsl->fn_fslport == 0) + if (nfsl->fn_fslport == 0 || nfsl->fn_fslport == NFS_PORT) printf(" NFS fls_server:\t\t%s\n", nfsl->fn_fslhost); else printf(" NFS fls_server:\t\t%s:%u\n", nfsl->fn_fslhost,