Author: chs
Date: Mon Mar  9 15:55:13 2020
New Revision: 358812
URL: https://svnweb.freebsd.org/changeset/base/358812

Log:
  Use the devfs vnode rather than the mntfs vnode for permissions checks.
  I missed this one in r358714.
  
  Reported by:  pho
  Reviewed by:  mckusick
  Approved by:  imp (mentor)
  Sponsored by: Netflix

Modified:
  head/sys/ufs/ffs/ffs_suspend.c

Modified: head/sys/ufs/ffs/ffs_suspend.c
==============================================================================
--- head/sys/ufs/ffs/ffs_suspend.c      Mon Mar  9 15:48:53 2020        
(r358811)
+++ head/sys/ufs/ffs/ffs_suspend.c      Mon Mar  9 15:55:13 2020        
(r358812)
@@ -196,10 +196,10 @@ ffs_susp_suspend(struct mount *mp)
         * device.  The permissions can change after we unlock the vnode;
         * it's harmless.
         */
-       vn_lock(ump->um_devvp, LK_EXCLUSIVE | LK_RETRY);
-       error = VOP_ACCESS(ump->um_devvp, VREAD | VWRITE,
+       vn_lock(ump->um_odevvp, LK_EXCLUSIVE | LK_RETRY);
+       error = VOP_ACCESS(ump->um_odevvp, VREAD | VWRITE,
            curthread->td_ucred, curthread);
-       VOP_UNLOCK(ump->um_devvp);
+       VOP_UNLOCK(ump->um_odevvp);
        if (error != 0)
                return (error);
 #ifdef MAC
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to