Re: [PATCH] Allow unattended nfs3/krb5 mounts

From: Jasper Spaans (jasper@vs19.net)
Date: Thu Jul 17 2003 - 14:04:28 EST


On Tue, Jul 15, 2003 at 11:26:05PM -0700, Frank Cusack wrote:
> The comment in nfs_get_root() basically describes the patch:
>
> Some authentication types (gss/krb5, most notably)
> are such that root won't be able to present a
> credential for GETATTR (ie, getroot()).
[..]
> Does this patch look reasonable? It works in my environment, against
> a netapp server (with the rpcsec_gss patch I provided earlier).

The way this patch was imported in bk-cvs causes a compile-time failure
in fs/nfs/inode.c;

Below is the -what seems to me- obvious fix.

One comment though wrt this code: it seems both the spellings 'flavor' and
'flavour' are used in this piece of code which is somewhat confusing. Should
that be fixed?

Index: fs/nfs/inode.c
===================================================================
RCS file: /home/cvs/linux-2.5/fs/nfs/inode.c,v
retrieving revision 1.79
diff -u -r1.79 inode.c
--- CVS/fs/nfs/inode.c 17 Jul 2003 17:28:19 -0000 1.79
+++ linux-2.5/fs/nfs/inode.c 17 Jul 2003 17:40:29 -0000
@@ -1441,7 +1441,7 @@
         if ((server->idmap = nfs_idmap_new(server)) == NULL)
                 printk(KERN_WARNING "NFS: couldn't start IDmap\n");
 
- err = nfs_sb_init(sb);
+ err = nfs_sb_init(sb, authflavour);
         if (err == 0)
                 return 0;
         rpciod_down();

VrGr,

-- 
Jasper Spaans                 http://jsp.vs19.net/contact/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Wed Jul 23 2003 - 22:00:30 EST