Changeset 2222 for trunk/eraser/Eraser.DefaultPlugins/ErasureTargets/FileSystemObjectErasureTarget.cs

Timestamp:

7/10/2010 8:36:36 AM (23 months ago)

Author:

lowjoel

Message:

Define proper behaviour when a directory reparse point (symbolic link/junction) is encountered. We should not touch the target of the reparse point, instead, just securely remove the reparse point. This also fixes a crash when a reparse point to the current directory is encountered (i.e. a folder referring to its immediate parent)

File:

• trunk/eraser/Eraser.DefaultPlugins/ErasureTargets/FileSystemObjectErasureTarget.cs (modified) (1 diff)

trunk/eraser/Eraser.DefaultPlugins/ErasureTargets/FileSystemObjectErasureTarget.cs

@@ -91 +91 @@
         /// </summary>
         /// <param name="info">The directory to look files in.</param>
-        /// <returns>A list of files found in the directory matching the IncludeMask
-        /// property.</returns>
+        /// <returns>A list of files found in the directory.</returns>
+        /// <remarks>This function does not recurse into directories which are
+        /// reparse points.</remarks>
         protected static FileInfo[] GetFiles(DirectoryInfo info)
         {
             List<FileInfo> result = new List<FileInfo>();
-            if (info.Exists)
+            if (info.Exists && (info.Attributes & FileAttributes.ReparsePoint) == 0)
             {
                 try