[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Cluster-devel] [GFS2 patch] NFS filehandle check



Steve,

I removed the printk.....

Wendy
---


File handle checking error found in '07 NFS connectathon. The fh_type
and fh_len are not necessarily identical. Some of the client machines
could fail mount with stale filehandle without this patch. 

Signed-off-by: S. Wendy Cheng <wcheng redhat com>

ops_export.c |    5 +----
1 files changed, 1 insertion(+), 4 deletions(-)

--- linux-feb-17/fs/gfs2/ops_export.c	2007-02-17 20:07:16.000000000 -0500
+++ up-kernel/fs/gfs2/ops_export.c	2007-02-23 00:03:11.000000000 -0500
@@ -38,14 +38,11 @@ static struct dentry *gfs2_decode_fh(str
 	struct gfs2_fh_obj fh_obj;
 	struct gfs2_inum_host *this, parent;
 
-	if (fh_type != fh_len)
-		return NULL;
-
 	this 		= &fh_obj.this;
 	fh_obj.imode 	= DT_UNKNOWN;
 	memset(&parent, 0, sizeof(struct gfs2_inum));
 
-	switch (fh_type) {
+	switch (fh_len) {
 	case GFS2_LARGE_FH_SIZE:
 		parent.no_formal_ino = ((u64)be32_to_cpu(fh[4])) << 32;
 		parent.no_formal_ino |= be32_to_cpu(fh[5]);

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]