From: Philip Homburg <philip@cs.vu.nl>
Date: Fri, 10 Aug 2007 13:01:38 +0000 (+0000)
Subject: Better recovery when req_readsuper fails.
X-Git-Tag: v3.1.4~332
X-Git-Url: http://zhaoyanbai.com/repos/%22/xml/v3/zones/static/man.7.txt?a=commitdiff_plain;h=06e1f0da615c8fa224bfd4d2cd2b7b15ea5b2af7;p=minix.git

Better recovery when req_readsuper fails.
---

diff --git a/servers/vfs/mount.c b/servers/vfs/mount.c
index 7a96ef1c9..193ee40e1 100644
--- a/servers/vfs/mount.c
+++ b/servers/vfs/mount.c
@@ -100,10 +100,10 @@ PRIVATE int mount_fs(endpoint_t fs_e)
   struct dmap *dp;
   dev_t dev;
   message m;
-  struct vnode *vp, *root_node, *Xmounted_on, *bspec;
+  struct vnode *vp, *root_node, *mounted_on, *bspec;
   struct vmnt *vmp, *vmp2;
   char *label;
-  struct node_details resX;
+  struct node_details res;
   
   /* Only the super-user may do MOUNT. */
   if (!super_user) return(EPERM);
@@ -185,7 +185,7 @@ PRIVATE int mount_fs(endpoint_t fs_e)
 	}
 
 	/* Request lookup */
-	r = lookup_vp(0 /*flags*/, 0 /*!use_realuid*/, &Xmounted_on);
+	r = lookup_vp(0 /*flags*/, 0 /*!use_realuid*/, &mounted_on);
 	if (r != OK) return r;
 
 	if (vp->v_ref_count != 1)
@@ -196,10 +196,10 @@ PRIVATE int mount_fs(endpoint_t fs_e)
 	}
 
 	/* Issue mountpoint request */
-	r = req_mountpoint(Xmounted_on->v_fs_e, Xmounted_on->v_inode_nr);
+	r = req_mountpoint(mounted_on->v_fs_e, mounted_on->v_inode_nr);
 	if (r != OK)
 	{
-		put_vnode(Xmounted_on);
+		put_vnode(mounted_on);
 		printf("vfs:mount_fs: req_mountpoint_s failed with %d\n", r);
 		return r;
 	}
@@ -209,7 +209,7 @@ PRIVATE int mount_fs(endpoint_t fs_e)
 
 	/* File types may not conflict. */
 	if (r == OK) {
-		mdir = ((Xmounted_on->v_mode & I_TYPE) == I_DIRECTORY); 
+		mdir = ((mounted_on->v_mode & I_TYPE) == I_DIRECTORY); 
 		/* TRUE iff dir */
 		rdir = ((root_node->v_mode & I_TYPE) == I_DIRECTORY);
 		if (!mdir && rdir) r = EISDIR;
@@ -217,14 +217,14 @@ PRIVATE int mount_fs(endpoint_t fs_e)
 
 	/* If error, return the mount point. */
 	if (r != OK) {
-		put_vnode(Xmounted_on);
+		put_vnode(mounted_on);
 
 		return(r);
 	}
 
 	/* Nothing else can go wrong.  Perform the mount. */
 	put_vnode(vmp->m_mounted_on);
-	vmp->m_mounted_on = Xmounted_on;
+	vmp->m_mounted_on = mounted_on;
 	vmp->m_flags = m_in.rd_only;
 	allow_newroot = 0;              /* The root is now fixed */
 
@@ -246,14 +246,14 @@ PRIVATE int mount_fs(endpoint_t fs_e)
 	printf("vfs:mount_fs: mount point at '%s'\n", user_fullpath);
 #endif
 
-	r = lookup_vp(0 /*flags*/, 0 /*!use_realuid*/, &Xmounted_on);
+	r = lookup_vp(0 /*flags*/, 0 /*!use_realuid*/, &mounted_on);
 	if (r != OK)
 		return r;
 
 	/* Issue mountpoint request */
-	r = req_mountpoint(Xmounted_on->v_fs_e, Xmounted_on->v_inode_nr);
+	r = req_mountpoint(mounted_on->v_fs_e, mounted_on->v_inode_nr);
 	if (r != OK) {
-		put_vnode(Xmounted_on);
+		put_vnode(mounted_on);
 		printf("vfs:mount_fs: req_mountpoint_s failed with %d\n", r);
 		return r;
 	}
@@ -282,24 +282,26 @@ PRIVATE int mount_fs(endpoint_t fs_e)
 #endif
 
   /* Issue request */
-  r = req_readsuper(fs_e, label, dev, m_in.rd_only, isroot, &resX);
+  r = req_readsuper(fs_e, label, dev, m_in.rd_only, isroot, &res);
   if (r != OK) {
-      return r;
+	put_vnode(mounted_on);
+	printf("vfs:mount_fs: req_readsuper failed with %d\n", r);
+	return r;
   }
 
   /* Fill in root node's fields */
-  root_node->v_fs_e = resX.fs_e;
-  root_node->v_inode_nr = resX.inode_nr;
-  root_node->v_mode = resX.fmode;
-  root_node->v_uid = resX.uid;
-  root_node->v_gid = resX.gid;
-  root_node->v_size = resX.fsize;
+  root_node->v_fs_e = res.fs_e;
+  root_node->v_inode_nr = res.inode_nr;
+  root_node->v_mode = res.fmode;
+  root_node->v_uid = res.uid;
+  root_node->v_gid = res.gid;
+  root_node->v_size = res.fsize;
   root_node->v_sdev = NO_DEV;
   root_node->v_fs_count = 1;
   root_node->v_ref_count = 1;
 
   /* Fill in max file size and blocksize for the vmnt */
-  vmp->m_fs_e = resX.fs_e;
+  vmp->m_fs_e = res.fs_e;
   vmp->m_dev = dev;
   vmp->m_flags = m_in.rd_only;
   vmp->m_driver_e = dp->dmap_driver;
@@ -342,14 +344,14 @@ PRIVATE int mount_fs(endpoint_t fs_e)
 
   /* File types may not conflict. */
   if (r == OK) {
-      mdir = ((Xmounted_on->v_mode & I_TYPE) == I_DIRECTORY);/* TRUE iff dir */
+      mdir = ((mounted_on->v_mode & I_TYPE) == I_DIRECTORY);/* TRUE iff dir */
       rdir = ((root_node->v_mode & I_TYPE) == I_DIRECTORY);
       if (!mdir && rdir) r = EISDIR;
   }
 
   /* If error, return the super block and both inodes; release the vmnt. */
   if (r != OK) {
-      put_vnode(Xmounted_on);
+      put_vnode(mounted_on);
       put_vnode(root_node);
 
       vmp->m_dev = NO_DEV;
@@ -357,7 +359,7 @@ PRIVATE int mount_fs(endpoint_t fs_e)
   }
 
   /* Nothing else can go wrong.  Perform the mount. */
-  vmp->m_mounted_on = Xmounted_on;
+  vmp->m_mounted_on = mounted_on;
   vmp->m_root_node = root_node;
 
   /* The root is now fixed */