Author: kib
Date: Tue Oct 19 08:55:31 2010
New Revision: 214049
URL: http://svn.freebsd.org/changeset/base/214049

Log:
  When readdirplus() is handled on the exported filesystem that does
  not support VFS_VGET, like msdosfs, do not call VOP_LOOKUP() for
  dotdot on the root directory. Our filesystems expect that VFS handles
  dotdot lookups on root on its own.
  
  Reported and tested by:       kevlo
  MFC after:   2 weeks

Modified:
  head/sys/nfsserver/nfs_serv.c

Modified: head/sys/nfsserver/nfs_serv.c
==============================================================================
--- head/sys/nfsserver/nfs_serv.c       Tue Oct 19 00:20:00 2010        
(r214048)
+++ head/sys/nfsserver/nfs_serv.c       Tue Oct 19 08:55:31 2010        
(r214049)
@@ -3036,7 +3036,7 @@ nfsrv_readdirplus(struct nfsrv_descript 
        struct iovec iv;
        struct vattr va, at, *vap = &va;
        struct nfs_fattr *fp;
-       int len, nlen, rem, xfer, tsiz, i, error = 0, getret = 1;
+       int len, nlen, rem, xfer, tsiz, i, error = 0, error1, getret = 1;
        int siz, cnt, fullsiz, eofflag, rdonly, dirlen, ncookies;
        u_quad_t off, toff, verf;
        u_long *cookies = NULL, *cookiep; /* needs to be int64_t or off_t */
@@ -3240,24 +3240,25 @@ again:
                                }
                                if (!VOP_ISLOCKED(vp))
                                        vn_lock(vp, LK_SHARED | LK_RETRY);
-                               if (VOP_LOOKUP(vp, &nvp, &cn) != 0)
+                               if ((vp->v_vflag & VV_ROOT) != 0 &&
+                                   (cn.cn_flags & ISDOTDOT) != 0) {
+                                       vref(vp);
+                                       nvp = vp;
+                               } else if (VOP_LOOKUP(vp, &nvp, &cn) != 0)
                                        goto invalid;
                        }
 
                        bzero((caddr_t)nfhp, NFSX_V3FH);
                        nfhp->fh_fsid = nvp->v_mount->mnt_stat.f_fsid;
-                       if (VOP_VPTOFH(nvp, &nfhp->fh_fid)) {
+                       if ((error1 = VOP_VPTOFH(nvp, &nfhp->fh_fid)) == 0)
+                               error1 = VOP_GETATTR(nvp, vap, cred);
+                       if (vp == nvp)
+                               vunref(nvp);
+                       else
                                vput(nvp);
-                               nvp = NULL;
-                               goto invalid;
-                       }
-                       if (VOP_GETATTR(nvp, vap, cred)) {
-                               vput(nvp);
-                               nvp = NULL;
-                               goto invalid;
-                       }
-                       vput(nvp);
                        nvp = NULL;
+                       if (error1 != 0)
+                               goto invalid;
 
                        /*
                         * If either the dircount or maxcount will be
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to