svn commit: r295714 - stable/10/sys/fs/ext2fs

[Pedro F. Giffuni]

Author: pfg
Date: Wed Feb 17 19:09:06 2016
New Revision: 295714
URL: https://svnweb.freebsd.org/changeset/base/295714

Log:
  MFC	r295616:
  ext2fs:	Remove panics for rename() race conditions.
  
  Sync with r84642 from UFS:
  
  The panics are inappropriate because the IN_RENAME flag only fixes a
  few of the huge number of race conditions that can result in the
  source path becoming invalid even prior to the VOP_RENAME() call.
  
  Approved by:	re (glebius)

Modified:
  stable/10/sys/fs/ext2fs/ext2_vnops.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/fs/ext2fs/ext2_vnops.c
==============================================================================
--- stable/10/sys/fs/ext2fs/ext2_vnops.c	Wed Feb 17 18:41:55 2016	(r295713)
+++ stable/10/sys/fs/ext2fs/ext2_vnops.c	Wed Feb 17 19:09:06 2016	(r295714)
@@ -985,10 +985,10 @@ abortit:
 		dp = VTOI(fdvp);
 	} else {
 		/*
-		 * From name has disappeared.
+		 * From name has disappeared.  IN_RENAME is not sufficient
+		 * to protect against directory races due to timing windows,
+		 * so we can't panic here.
 		 */
-		if (doingdirectory)
-			panic("ext2_rename: lost dir entry");
 		vrele(ap->a_fvp);
 		return (0);
 	}
@@ -1003,8 +1003,11 @@ abortit:
 	 * rename.
 	 */
 	if (xp != ip) {
-		if (doingdirectory)
-			panic("ext2_rename: lost dir entry");
+		/*
+		 * From name resolves to a different inode.  IN_RENAME is
+		 * not sufficient protection against timing window races
+		 * so we can't panic here.
+		 */
 	} else {
 		/*
 		 * If the source is a directory with a