Re: [PATCH] NFS: initialize default port in kernel mount client

[Chuck Lever <chuck.lever@oracle.com>]

[-- Attachment #1: Type: text/plain, Size: 1119 bytes --]

Hi James-

James Lentini wrote:
> If no mount server port number is specified, this change to the kernel 
> mount client
> 
> http://linux-nfs.org/cgi-bin/gitweb.cgi?p=nfs-2.6.git;a=commitdiff;h=209ce98
> 06996537eceaf07d95def0e9300316c84
> 
> inadvertently allows the NFS server's port number to be the used as 
> the mount server's port number. If the user specifies an NFS server 
> port (-o port=x), the mount will fail.
> 
> The fix below sets the mount server's port to 0 if no mount server 
> port is specified by the user.
> 
> Signed-off-by: James Lentini <jlentini@netapp.com>
> --
> 
>  super.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> Index: nfs-2.6/fs/nfs/super.c
> ===================================================================
> --- nfs-2.6/fs/nfs/super.c
> +++ nfs-2.6/fs/nfs/super.c
> @@ -1006,6 +1006,8 @@ static int nfs_try_mount(struct nfs_pars
>  		sin = args->nfs_server.address;
>  	if (args->mount_server.port != 0)
>  		sin.sin_port = htons(args->mount_server.port);
> +	else
> +		sin.sin_port = htons(0);

Why not just set sin.sin_port = args->mount_server.port unconditionally?

[-- Attachment #2: chuck.lever.vcf --]
[-- Type: text/x-vcard, Size: 290 bytes --]

begin:vcard
fn:Chuck Lever
n:Lever;Chuck
org:Oracle Corporation;Corporate Architecture: Linux Projects Group
adr:;;1015 Granger Avenue;Ann Arbor;MI;48104;USA
title:Principal Member of Staff
tel;work:+1 248 614 5091
x-mozilla-html:FALSE
url:http://oss.oracle.com/~cel
version:2.1
end:vcard


[-- Attachment #3: Type: text/plain, Size: 228 bytes --]

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

[-- Attachment #4: Type: text/plain, Size: 140 bytes --]

_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs