Changeset 69720 in vbox for trunk

Timestamp:

Nov 16, 2017 4:18:19 PM (7 years ago)

Author:

vboxsync

Message:

IPRT: More work on directory relative APIs (NT mainly) and VFS; introducing RTRmDir (test) tool.

• Added RTVfsDirRemoveDir
• Native NT implementation of RTDirRelRemoveDir.
• Improved the RTFS_TYPE_DIRECTORY case of rtVfsStdDir_UnlinkEntry.
• Added two new status code conversions to RTErrConvertFromNtStatus.

Location:

trunk

Files:

• include/iprt/mangling.h (modified) (1 diff)
• include/iprt/vfs.h (modified) (1 diff)
• src/VBox/Runtime/common/vfs/vfsbase.cpp (modified) (1 diff)
• src/VBox/Runtime/common/vfs/vfsstddir.cpp (modified) (1 diff)
• src/VBox/Runtime/nt/RTErrConvertFromNtStatus.cpp (modified) (1 diff)
• src/VBox/Runtime/r3/nt/dirrel-r3-nt.cpp (modified) (3 diffs)
• src/VBox/Runtime/tools/Makefile.kmk (modified) (1 diff)
• src/VBox/Runtime/tools/RTMkDir.cpp (modified) (1 diff)
• src/VBox/Runtime/tools/RTRmDir.cpp (copied) (copied from trunk/src/VBox/Runtime/tools/RTMkDir.cpp ) (19 diffs)

trunk/include/iprt/mangling.h

@@ -2438 +2438 @@
 # define RTVfsDirQueryPathInfo                          RT_MANGLER(RTVfsDirQueryPathInfo)
 # define RTVfsDirReadEx                                 RT_MANGLER(RTVfsDirReadEx)
+# define RTVfsDirRemoveDir                              RT_MANGLER(RTVfsDirRemoveDir)
 # define RTVfsFileFlush                                 RT_MANGLER(RTVfsFileFlush)
 # define RTVfsFileFromBuffer                            RT_MANGLER(RTVfsFileFromBuffer)

trunk/include/iprt/vfs.h

@@ -528 +528 @@
 
 /**
+ * Removes a directory relative to @a hVfsDir.
+ *
+ * @returns IPRT status code.
+ * @param   hVfsDir         The VFS directory to start walking the @a pszRelPath
+ *                          relative to.
+ * @param   pszRelPath      The path to the directory that should be removed.
+ * @param   fFlags          Reserved, MBZ.
+ */
+RTDECL(int) RTVfsDirRemoveDir(RTVFSDIR hVfsDir, const char *pszRelPath, uint32_t fFlags);
+
+/**
  * Reads the next entry in the directory returning extended information.
  *

trunk/src/VBox/Runtime/common/vfs/vfsbase.cpp

@@ -2540 +2540 @@
 
 
+RTDECL(int) RTVfsDirRemoveDir(RTVFSDIR hVfsDir, const char *pszRelPath, uint32_t fFlags)
+{
+    /*
+     * Validate input.
+     */
+    RTVFSDIRINTERNAL *pThis = hVfsDir;
+    AssertPtrReturn(pThis, VERR_INVALID_HANDLE);
+    AssertReturn(pThis->uMagic == RTVFSDIR_MAGIC, VERR_INVALID_HANDLE);
+    AssertPtrReturn(pszRelPath, VERR_INVALID_POINTER);
+    AssertReturn(!fFlags, VERR_INVALID_FLAGS);
+
+    /*
+     * Parse the path, it's always relative to the given directory.
+     */
+    PRTVFSPARSEDPATH pPath;
+    int rc = RTVfsParsePathA(pszRelPath, NULL, &pPath);
+    if (RT_SUCCESS(rc))
+    {
+        if (pPath->cComponents > 0)
+        {
+            /*
+             * Tranverse the path, resolving the parent node, not checking for symbolic
+             * links in the final element, and ask the directory to remove the subdir.
+             */
+            RTVFSDIRINTERNAL *pVfsParentDir;
+            rc = rtVfsDirTraverseToParent(pThis, pPath, RTPATH_F_ON_LINK, &pVfsParentDir);
+            if (RT_SUCCESS(rc))
+            {
+                const char *pszEntryName = &pPath->szPath[pPath->aoffComponents[pPath->cComponents - 1]];
+
+                RTVfsLockAcquireWrite(pVfsParentDir->Base.hLock);
+                rc = pVfsParentDir->pOps->pfnUnlinkEntry(pVfsParentDir->Base.pvThis, pszEntryName, RTFS_TYPE_DIRECTORY);
+                RTVfsLockReleaseWrite(pVfsParentDir->Base.hLock);
+
+                RTVfsDirRelease(pVfsParentDir);
+            }
+        }
+        else
+            rc = VERR_PATH_ZERO_LENGTH;
+        RTVfsParsePathFree(pPath);
+    }
+    return rc;
+}
+
+
+
 RTDECL(int) RTVfsDirReadEx(RTVFSDIR hVfsDir, PRTDIRENTRYEX pDirEntry, size_t *pcbDirEntry, RTFSOBJATTRADD enmAddAttr)
 {

trunk/src/VBox/Runtime/common/vfs/vfsstddir.cpp

@@ -443 +443 @@
     if (fType != 0)
     {
+        if (fType == RTFS_TYPE_DIRECTORY)
+            return RTDirRelDirRemove(pThis->hDir, pszEntry);
+
         RTFSOBJINFO ObjInfo;
         int rc = rtVfsStdDir_QueryEntryInfo(pThis, pszEntry, &ObjInfo, RTFSOBJATTRADD_NOTHING);

trunk/src/VBox/Runtime/nt/RTErrConvertFromNtStatus.cpp

@@ -81 +81 @@
         case STATUS_BAD_NETWORK_PATH:       return VERR_NET_PATH_NOT_FOUND;
         case STATUS_NOT_A_DIRECTORY:        return VERR_NOT_A_DIRECTORY;
+        case STATUS_DIRECTORY_NOT_EMPTY:    return VERR_DIR_NOT_EMPTY;
+        case STATUS_SHARING_VIOLATION:      return VERR_SHARING_VIOLATION;
 
         case STATUS_UNEXPECTED_NETWORK_ERROR:

trunk/src/VBox/Runtime/r3/nt/dirrel-r3-nt.cpp

@@ -217 +217 @@
 
 
-/**
- * Creates a directory relative to @a hDir.
- *
- * @returns IPRT status code.
- * @param   hDir            The directory @a pszRelPath is relative to.
- * @param   pszRelPath      The relative path to the directory to create.
- * @param   fMode           The mode of the new directory.
- * @param   fCreate         Create flags, RTDIRCREATE_FLAGS_XXX.
- * @param   phSubDir        Where to return the handle of the created directory.
- *                          Optional.
- *
- * @sa      RTDirCreate
- */
 RTDECL(int) RTDirRelDirCreate(PRTDIR hDir, const char *pszRelPath, RTFMODE fMode, uint32_t fCreate, PRTDIR *phSubDir)
 {
@@ -321 +308 @@
 
 
-/**
- * Removes a directory relative to @a hDir if empty.
- *
- * @returns IPRT status code.
- * @param   hDir            The directory @a pszRelPath is relative to.
- * @param   pszRelPath      The relative path to the directory to remove.
- *
- * @sa      RTDirRemove
- */
 RTDECL(int) RTDirRelDirRemove(PRTDIR hDir, const char *pszRelPath)
 {
@@ -336 +314 @@
     AssertReturn(pThis->u32Magic == RTDIR_MAGIC, VERR_INVALID_HANDLE);
 
-
-    char szPath[RTPATH_MAX];
-    int rc = rtDirRelBuildFullPath(pThis, szPath, sizeof(szPath), pszRelPath);
-    if (RT_SUCCESS(rc))
-    {
-RTAssertMsg2("DBG: RTDirRelDirRemove(%s)...\n", szPath);
-        rc = RTDirRemove(szPath);
+    /*
+     * Convert and normalize the path.
+     */
+    UNICODE_STRING NtName;
+    HANDLE hRoot = pThis->hDir;
+    int rc = RTNtPathRelativeFromUtf8(&NtName, &hRoot, pszRelPath, RTDIRREL_NT_GET_ASCENT(pThis),
+                                      pThis->enmInfoClass == FileMaximumInformation);
+    if (RT_SUCCESS(rc))
+    {
+        HANDLE              hSubDir = RTNT_INVALID_HANDLE_VALUE;
+        IO_STATUS_BLOCK     Ios     = RTNT_IO_STATUS_BLOCK_INITIALIZER;
+        OBJECT_ATTRIBUTES   ObjAttr;
+        InitializeObjectAttributes(&ObjAttr, &NtName, 0 /*fAttrib*/, hRoot, NULL);
+
+        NTSTATUS rcNt = NtCreateFile(&hSubDir,
+                                     DELETE | SYNCHRONIZE,
+                                     &ObjAttr,
+                                     &Ios,
+                                     NULL /*AllocationSize*/,
+                                     FILE_ATTRIBUTE_NORMAL,
+                                     FILE_SHARE_DELETE | FILE_SHARE_READ | FILE_SHARE_WRITE,
+                                     FILE_OPEN,
+                                     FILE_DIRECTORY_FILE | FILE_OPEN_FOR_BACKUP_INTENT | FILE_SYNCHRONOUS_IO_NONALERT | FILE_OPEN_REPARSE_POINT,
+                                     NULL /*EaBuffer*/,
+                                     0 /*EaLength*/);
+        if (NT_SUCCESS(rcNt))
+        {
+            FILE_DISPOSITION_INFORMATION DispInfo;
+            DispInfo.DeleteFile = TRUE;
+            RTNT_IO_STATUS_BLOCK_REINIT(&Ios);
+            rcNt = NtSetInformationFile(hSubDir, &Ios, &DispInfo, sizeof(DispInfo), FileDispositionInformation);
+
+            NTSTATUS rcNt2 = NtClose(hSubDir);
+            if (!NT_SUCCESS(rcNt2) && NT_SUCCESS(rcNt))
+                rcNt = rcNt2;
+        }
+
+        if (NT_SUCCESS(rcNt))
+            rc = VINF_SUCCESS;
+        else
+            rc = RTErrConvertFromNtStatus(rcNt);
+
+        RTNtPathFree(&NtName, NULL);
     }
     return rc;

trunk/src/VBox/Runtime/tools/Makefile.kmk

@@ -118 +118 @@
  RTMkDir_SOURCES = RTMkDir.cpp
 
+ # RTRmDir - our mkdir clone.
+ PROGRAMS += RTRmDir
+ RTRmDir_TEMPLATE = VBoxR3Tool
+ RTRmDir_SOURCES = RTRmDir.cpp
+
  # RTShutdown - similar (but not identical) to a typical unix shutdown command.
  PROGRAMS += RTShutdown

trunk/src/VBox/Runtime/tools/RTMkDir.cpp

@@ -108 +108 @@
                 if (RT_FAILURE(rc))
                     RTVfsChainMsgError("RTVfsChainOpenDir", pszSpec, rc, offError, &ErrInfo.Core);
+                else if (!pszFinalPath)
+                    pszFinalPath = RTStrEnd(pszSpec, RTSTR_MAX);
             }
             else if (!RTPathStartsWithRoot(pszFinalPath))

trunk/src/VBox/Runtime/tools/RTRmDir.cpp

@@ -1 +1 @@
 /* $Id$ */
 /** @file
- * IPRT - Creates directory.
+ * IPRT - Removes directory.
  */
 
@@ -44 +44 @@
 *   Structures and Typedefs                                                                                                      *
 *********************************************************************************************************************************/
-typedef struct RTCMDMKDIROPTS
+typedef struct RTCMDRMDIROPTS
 {
     /** -v, --verbose */
@@ -50 +50 @@
     /** -p, --parents */
     bool        fParents;
+    /** Don't fail if directories that aren't empty. */
+    bool        fIgnoreNotEmpty;
+    /** Don't fail a directory doesn't exist (i.e. has already been removed). */
+    bool        fIgnoreNonExisting;
     /** Whether to always use the VFS chain API (for testing). */
     bool        fAlwaysUseChainApi;
-    /** Directory creation flags (RTDIRCREATE_FLAGS_XXX).   */
-    uint32_t    fCreateFlags;
-    /** The directory mode. */
-    RTFMODE     fMode;
-} RTCMDMKDIROPTS;
+} RTCMDRMDIROPTS;
 
 
@@ -66 +66 @@
  * @param   pszDir              The path to the new directory.
  */
-static int rtCmdMkDirOneWithParents(RTCMDMKDIROPTS const *pOpts, const char *pszDir)
-{
+static int rtCmdRmDirOneWithParents(RTCMDRMDIROPTS const *pOpts, const char *pszDir)
+{
+    /* We need a copy we can work with here. */
+    char *pszCopy = RTStrDup(pszDir);
+    if (!pszCopy)
+        return RTMsgErrorExitFailure("Out of string memory!");
+
     int rc;
     if (!pOpts->fAlwaysUseChainApi && !RTVfsChainIsSpec(pszDir) )
     {
-        /*
-         * Use the API for doing the entire job.  Unfortuantely, this means we
-         * can't be very  verbose about what we're doing.
-         */
-        rc = RTDirCreateFullPath(pszDir, pOpts->fMode);
-        if (RT_FAILURE(rc))
-            RTMsgError("Failed to create directory '%s' (or a parent): %Rrc", pszDir, rc);
-        else if (pOpts->fVerbose)
-            RTPrintf("%s\n", pszDir);
+        size_t cchCopy = strlen(pszCopy);
+        do
+        {
+            rc = RTDirRemove(pszCopy);
+            if (RT_SUCCESS(rc))
+            {
+                if (pOpts->fVerbose)
+                    RTPrintf("%s\n", pszCopy);
+            }
+            else if ((rc == VERR_PATH_NOT_FOUND || rc == VERR_FILE_NOT_FOUND) && pOpts->fIgnoreNonExisting)
+                rc = VINF_SUCCESS;
+            else
+            {
+                if ((rc == VERR_DIR_NOT_EMPTY || rc == VERR_SHARING_VIOLATION) && pOpts->fIgnoreNotEmpty)
+                    rc = VINF_SUCCESS;
+                else
+                    RTMsgError("Failed to remove directory '%s': %Rrc", pszCopy, rc);
+                break;
+            }
+
+            /* Strip off a component. */
+            while (cchCopy > 0 && RTPATH_IS_SLASH(pszCopy[cchCopy - 1]))
+                cchCopy--;
+            while (cchCopy > 0 && !RTPATH_IS_SLASH(pszCopy[cchCopy - 1]))
+                cchCopy--;
+            while (cchCopy > 0 && RTPATH_IS_SLASH(pszCopy[cchCopy - 1]))
+                cchCopy--;
+            pszCopy[cchCopy] = '\0';
+        } while (cchCopy > 0);
     }
     else
@@ -86 +111 @@
          * Strip the final path element from the pszDir spec.
          */
-        char *pszCopy = RTStrDup(pszDir);
-        if (!pszCopy)
-            return RTMsgErrorExitFailure("Out of string memory!");
-
         char       *pszFinalPath;
         char       *pszSpec;
@@ -96 +117 @@
         if (RT_SUCCESS(rc))
         {
-            const char * const pszFullFinalPath = pszFinalPath;
-
             /*
              * Open the root director/whatever.
              */
             RTERRINFOSTATIC ErrInfo;
-            RTVFSDIR hVfsCurDir;
+            RTVFSDIR hVfsBaseDir;
             if (pszSpec)
             {
-                rc = RTVfsChainOpenDir(pszSpec, 0 /*fOpen*/, &hVfsCurDir, &offError, RTErrInfoInitStatic(&ErrInfo));
+                rc = RTVfsChainOpenDir(pszSpec, 0 /*fOpen*/, &hVfsBaseDir, &offError, RTErrInfoInitStatic(&ErrInfo));
                 if (RT_FAILURE(rc))
                     RTVfsChainMsgError("RTVfsChainOpenDir", pszSpec, rc, offError, &ErrInfo.Core);
+                else if (!pszFinalPath)
+                    pszFinalPath = RTStrEnd(pszSpec, RTSTR_MAX);
             }
             else if (!RTPathStartsWithRoot(pszFinalPath))
             {
-                rc = RTVfsDirOpenNormal(".", 0 /*fOpen*/, &hVfsCurDir);
+                rc = RTVfsDirOpenNormal(".", 0 /*fOpen*/, &hVfsBaseDir);
                 if (RT_FAILURE(rc))
                     RTMsgError("Failed to open '.' (for %s): %Rrc", rc, pszFinalPath);
@@ -121 +142 @@
                 char const chSaved = *pszFinalPath;
                 *pszFinalPath = '\0';
-                rc = RTVfsDirOpenNormal(pszRoot, 0 /*fOpen*/, &hVfsCurDir);
+                rc = RTVfsDirOpenNormal(pszRoot, 0 /*fOpen*/, &hVfsBaseDir);
                 *pszFinalPath = chSaved;
                 if (RT_FAILURE(rc))
@@ -128 +149 @@
 
             /*
-             * Walk the path component by component.
+             * Walk the path component by component, starting at the end.
              */
-            while (RT_SUCCESS(rc))
-            {
-                /*
-                 * Strip leading slashes.
-                 */
-                while (RTPATH_IS_SLASH(*pszFinalPath))
-                    pszFinalPath++;
-                if (*pszFinalPath == '\0')
+            if (RT_SUCCESS(rc))
+            {
+                size_t cchFinalPath = strlen(pszFinalPath);
+                while (RT_SUCCESS(rc) && cchFinalPath > 0)
                 {
-                    RTVfsDirRelease(hVfsCurDir);
-                    break;
-                }
-
-                /*
-                 * Find the end of the next path component.
-                 */
-                size_t cchComponent = 0;
-                char   ch;
-                while (   (ch = pszFinalPath[cchComponent]) != '\0'
-                       && !RTPATH_IS_SLASH(ch))
-                    cchComponent++;
-
-                /*
-                 * Open or create the component.
-                 */
-                pszFinalPath[cchComponent] = '\0';
-                RTVFSDIR hVfsNextDir = NIL_RTVFSDIR;
-                for (uint32_t cTries = 0; cTries < 8; cTries++)
-                {
-                    /* Try open it. */
-                    rc = RTVfsDirOpenDir(hVfsCurDir, pszFinalPath, 0 /*fFlags*/, &hVfsNextDir);
+                    rc = RTVfsDirRemoveDir(hVfsBaseDir, pszFinalPath, 0 /*fFlags*/);
                     if (RT_SUCCESS(rc))
-                        break;
-                    if (   rc != VERR_FILE_NOT_FOUND
-                        && rc != VERR_PATH_NOT_FOUND)
                     {
-                        if (ch == '\0')
-                            RTMsgError("Failed opening directory '%s': %Rrc", pszDir, rc);
+                        if (pOpts->fVerbose)
+                            RTPrintf("%s\n", pszCopy);
+                    }
+                    else if ((rc == VERR_PATH_NOT_FOUND || rc == VERR_FILE_NOT_FOUND) && pOpts->fIgnoreNonExisting)
+                        rc = VINF_SUCCESS;
+                    else
+                    {
+                        if ((rc == VERR_DIR_NOT_EMPTY || rc == VERR_SHARING_VIOLATION) && pOpts->fIgnoreNotEmpty)
+                            rc = VINF_SUCCESS;
+                        else if (pszSpec)
+                            RTMsgError("Failed to remove directory '%s:%s': %Rrc", pszSpec, pszFinalPath, rc);
                         else
-                            RTMsgError("Failed opening dir '%s' (for creating '%s'): %Rrc", pszFullFinalPath, pszDir, rc);
+                            RTMsgError("Failed to remove directory '%s': %Rrc", pszFinalPath, rc);
                         break;
                     }
 
-                    /* Not found, so try create it. */
-                    rc = RTVfsDirCreateDir(hVfsCurDir, pszFinalPath, pOpts->fMode, pOpts->fCreateFlags, &hVfsNextDir);
-                    if (rc == VERR_ALREADY_EXISTS)
-                        continue; /* We lost a creation race, try again. */
-                    if (RT_SUCCESS(rc) && pOpts->fVerbose)
-                    {
-                        if (pszSpec)
-                            RTPrintf("%s:%s\n", pszSpec, pszFullFinalPath);
-                        else
-                            RTPrintf("%s\n", pszFullFinalPath);
-                    }
-                    else if (RT_FAILURE(rc))
-                    {
-                        if (ch == '\0')
-                            RTMsgError("Failed creating directory '%s': %Rrc", pszDir, rc);
-                        else
-                            RTMsgError("Failed creating dir '%s' (for '%s'): %Rrc", pszFullFinalPath, pszDir, rc);
-                    }
-                    break;
+                    /* Strip off a component. */
+                    while (cchFinalPath > 0 && RTPATH_IS_SLASH(pszFinalPath[cchFinalPath - 1]))
+                        cchFinalPath--;
+                    while (cchFinalPath > 0 && !RTPATH_IS_SLASH(pszFinalPath[cchFinalPath - 1]))
+                        cchFinalPath--;
+                    while (cchFinalPath > 0 && RTPATH_IS_SLASH(pszFinalPath[cchFinalPath - 1]))
+                        cchFinalPath--;
+                    pszFinalPath[cchFinalPath] = '\0';
                 }
-                pszFinalPath[cchComponent] = ch;
-
-                RTVfsDirRelease(hVfsCurDir);
-                hVfsCurDir = hVfsNextDir;
-                pszFinalPath += cchComponent;
+
+                RTVfsDirRelease(hVfsBaseDir);
             }
         }
         else
             RTVfsChainMsgError("RTVfsChainOpenParentDir", pszCopy, rc, offError, NULL);
-        RTStrFree(pszCopy);
-    }
+    }
+    RTStrFree(pszCopy);
     return RT_SUCCESS(rc) ? RTEXITCODE_SUCCESS : RTEXITCODE_FAILURE;
 }
@@ -209 +197 @@
 
 /**
- * Create one directory.
+ * Removes one directory.
  *
  * @returns exit code
@@ -215 +203 @@
  * @param   pszDir              The path to the new directory.
  */
-static RTEXITCODE rtCmdMkDirOne(RTCMDMKDIROPTS const *pOpts, const char *pszDir)
+static RTEXITCODE rtCmdRmDirOne(RTCMDRMDIROPTS const *pOpts, const char *pszDir)
 {
     int rc;
     if (!pOpts->fAlwaysUseChainApi && !RTVfsChainIsSpec(pszDir) )
-        rc = RTDirCreate(pszDir, pOpts->fMode, 0);
+        rc = RTDirRemove(pszDir);
     else
     {
@@ -229 +217 @@
         if (RT_SUCCESS(rc))
         {
-            rc = RTVfsDirCreateDir(hVfsDir, pszChild, pOpts->fMode, 0 /*fFlags*/, NULL);
+            rc = RTVfsDirRemoveDir(hVfsDir, pszChild, 0 /*fFlags*/);
             RTVfsDirRelease(hVfsDir);
         }
@@ -241 +229 @@
         return RTEXITCODE_SUCCESS;
     }
-    return RTMsgErrorExitFailure("Failed to create '%s': %Rrc", pszDir, rc);
+    if ((rc == VERR_DIR_NOT_EMPTY || rc == VERR_SHARING_VIOLATION) && pOpts->fIgnoreNotEmpty)
+        return RTEXITCODE_SUCCESS; /** @todo be verbose about this? */
+    if ((rc == VERR_PATH_NOT_FOUND || rc == VERR_FILE_NOT_FOUND) && pOpts->fIgnoreNonExisting)
+        return RTEXITCODE_SUCCESS; /** @todo be verbose about this? */
+    return RTMsgErrorExitFailure("Failed to remove '%s': %Rrc", pszDir, rc);
 }
 
 
-static RTEXITCODE RTCmdMkDir(unsigned cArgs,  char **papszArgs)
+static RTEXITCODE RTCmdRmDir(unsigned cArgs,  char **papszArgs)
 {
     /*
@@ -253 +245 @@
     {
         /* operations */
-        { "--mode",                     'm', RTGETOPT_REQ_UINT32 | RTGETOPT_FLAG_OCT },
         { "--parents",                  'p', RTGETOPT_REQ_NOTHING },
+        { "--ignore-fail-on-non-empty", 'F', RTGETOPT_REQ_NOTHING },
+        { "--ignore-non-existing",      'E', RTGETOPT_REQ_NOTHING },
         { "--always-use-vfs-chain-api", 'A', RTGETOPT_REQ_NOTHING },
-        { "--allow-content-indexing",   'i', RTGETOPT_REQ_NOTHING },
         { "--verbose",                  'v', RTGETOPT_REQ_NOTHING },
     };
@@ -266 +258 @@
         return RTMsgErrorExit(RTEXITCODE_FAILURE, "RTGetOpt failed: %Rrc", rc);
 
-    RTCMDMKDIROPTS Opts;
+    RTCMDRMDIROPTS Opts;
     Opts.fVerbose               = false;
     Opts.fParents               = false;
+    Opts.fIgnoreNotEmpty        = false;
+    Opts.fIgnoreNonExisting     = false;
     Opts.fAlwaysUseChainApi     = false;
-    Opts.fCreateFlags           = RTDIRCREATE_FLAGS_NOT_CONTENT_INDEXED_NOT_CRITICAL | RTDIRCREATE_FLAGS_NOT_CONTENT_INDEXED_SET;
-    Opts.fMode                  = 0775 | RTFS_TYPE_DIRECTORY | RTFS_DOS_DIRECTORY;
 
     RTGETOPTUNION ValueUnion;
@@ -279 +271 @@
         switch (rc)
         {
-            case 'm':
-                /** @todo DOS+NT attributes and symbolic notation. */
-                Opts.fMode &= ~07777;
-                Opts.fMode |= ValueUnion.u32 & 07777;
-                break;
-
             case 'p':
                 Opts.fParents = true;
@@ -297 +283 @@
                 break;
 
-            case 'i':
-                Opts.fCreateFlags &= ~RTDIRCREATE_FLAGS_NOT_CONTENT_INDEXED_SET;
-                Opts.fCreateFlags |= RTDIRCREATE_FLAGS_NOT_CONTENT_INDEXED_DONT_SET;
+            case 'E':
+                Opts.fIgnoreNonExisting = true;
+                break;
+
+            case 'F':
+                Opts.fIgnoreNotEmpty = true;
                 break;
 
@@ -305 +294 @@
                 RTPrintf("Usage: %s [options] <dir> [..]\n"
                          "\n"
+                         "Removes empty directories.\n"
+                         "\n"
                          "Options:\n"
-                         "  -m <mode>, --mode <mode>\n"
-                         "      The creation mode. Default is 0775.\n"
                          "  -p, --parent\n"
-                         "      Create parent directories too. Ignore any existing directories.\n"
+                         "      Remove specified parent directories too.\n"
+                         "  -F, --ignore-fail-on-non-empty\n"
+                         "      Do not fail if a directory is not empty, just ignore it.\n"
+                         "      This is really handy with the -p option.\n"
+                         "  -E, --ignore-non-existing\n"
+                         "      Do not fail if a specified directory is not there.\n"
                          "  -v, --verbose\n"
-                         "      Tell which directories get created.\n"
+                         "      Tell which directories get remove.\n"
                          "  -A, --always-use-vfs-chain-api\n"
                          "      Always use the VFS API.\n"
-                         "  -i, --allow-content-indexing\n"
-                         "      Don't set flags to disable context indexing on windows.\n"
                          , papszArgs[0]);
                 return RTEXITCODE_SUCCESS;
@@ -342 +334 @@
     {
         if (Opts.fParents)
-            rc = rtCmdMkDirOneWithParents(&Opts, ValueUnion.psz);
+            rc = rtCmdRmDirOneWithParents(&Opts, ValueUnion.psz);
         else
-            rc = rtCmdMkDirOne(&Opts, ValueUnion.psz);
+            rc = rtCmdRmDirOne(&Opts, ValueUnion.psz);
         if (RT_FAILURE(rc))
             rcExit = RTEXITCODE_FAILURE;
@@ -363 +355 @@
     if (RT_FAILURE(rc))
         return RTMsgInitFailure(rc);
-    return RTCmdMkDir(argc, argv);
+    return RTCmdRmDir(argc, argv);
 }