svn commit: r358812 - head/sys/ufs/ffs

Chuck Silvers chs at FreeBSD.org
Mon Mar 9 15:55:14 UTC 2020


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


More information about the svn-src-all mailing list