Changeset 69758 in vbox for trunk/src/VBox/Frontends/VBoxManage

Timestamp:

Nov 19, 2017 3:23:57 PM (7 years ago)

Author:

vboxsync

Message:

VBoxManageGuestCtrl.cpp,DnDURIList.cpp: Fix RTDIRENTRYTYPE_UNKNOWN problem by using RTDirReadExA. Untested.

File:

• trunk/src/VBox/Frontends/VBoxManage/VBoxManageGuestCtrl.cpp (modified) (7 diffs)

trunk/src/VBox/Frontends/VBoxManage/VBoxManageGuestCtrl.cpp

@@ -2232 +2232 @@
              * Enumerate the directory tree.
              */
+            size_t        cbDirEntry = 0;
+            PRTDIRENTRYEX pDirEntry  = NULL;
             while (RT_SUCCESS(vrc))
             {
-                RTDIRENTRY DirEntry;
-                vrc = RTDirRead(hDir, &DirEntry, NULL);
+                vrc = RTDirReadExA(hDir, &pDirEntry, &cbDirEntry, RTFSOBJATTRADD_NOTHING, 0);
                 if (RT_FAILURE(vrc))
                 {
@@ -2242 +2243 @@
                     break;
                 }
-                /** @todo r=bird: This ain't gonna work on most UNIX file systems because
-                 *        enmType is RTDIRENTRYTYPE_UNKNOWN.  This is clearly documented in
-                 *        RTDIRENTRY::enmType. For trunk, RTDirQueryUnknownType can be used. */
-                switch (DirEntry.enmType)
+
+                switch (pDirEntry->Info.Attr.fMode & RTFS_TYPE_MASK)
                 {
-                    case RTDIRENTRYTYPE_DIRECTORY:
+                    case RTFS_TYPE_DIRECTORY:
                     {
                         /* Skip "." and ".." entries. */
-                        if (   !strcmp(DirEntry.szName, ".")
-                            || !strcmp(DirEntry.szName, ".."))
+                        if (RTDirEntryExIsStdDotLink(pDirEntry))
                             break;
 
                         if (pContext->pCmdCtx->cVerbose)
-                            RTPrintf("Directory: %s\n", DirEntry.szName);
+                            RTPrintf("Directory: %s\n", pDirEntry->szName);
 
                         if (enmFlags & kGctlCopyFlags_Recursive)
@@ -2261 +2259 @@
                             char *pszNewSub = NULL;
                             if (pszSubDir)
-                                pszNewSub = RTPathJoinA(pszSubDir, DirEntry.szName);
+                                pszNewSub = RTPathJoinA(pszSubDir, pDirEntry->szName);
                             else
                             {
-                                pszNewSub = RTStrDup(DirEntry.szName);
+                                pszNewSub = RTStrDup(pDirEntry->szName);
                                 RTPathStripTrailingSlash(pszNewSub);
                             }
@@ -2281 +2279 @@
                     }
 
-                    case RTDIRENTRYTYPE_SYMLINK:
+                    case RTFS_TYPE_SYMLINK:
                         if (   (enmFlags & kGctlCopyFlags_Recursive)
                             && (enmFlags & kGctlCopyFlags_FollowLinks))
-                        {
-                            /* Fall through to next case is intentional. */
-                        }
+                        { /* Fall through to next case is intentional. */ }
                         else
                             break;
-
-                    case RTDIRENTRYTYPE_FILE:
+                        RT_FALL_THRU();
+
+                    case RTFS_TYPE_FILE:
                     {
                         if (   pszFilter
-                            && !RTStrSimplePatternMatch(pszFilter, DirEntry.szName))
+                            && !RTStrSimplePatternMatch(pszFilter, pDirEntry->szName))
                         {
                             break; /* Filter does not match. */
@@ -2299 +2296 @@
 
                         if (pContext->pCmdCtx->cVerbose)
-                            RTPrintf("File: %s\n", DirEntry.szName);
+                            RTPrintf("File: %s\n", pDirEntry->szName);
 
                         if (!fDirCreated)
                         {
                             char *pszDestDir;
-                            vrc = gctlCopyTranslatePath(pszSource, szCurDir,
-                                                        pszDest, &pszDestDir);
+                            vrc = gctlCopyTranslatePath(pszSource, szCurDir, pszDest, &pszDestDir);
                             if (RT_SUCCESS(vrc))
                             {
@@ -2317 +2313 @@
                         if (RT_SUCCESS(vrc))
                         {
-                            char *pszFileSource = RTPathJoinA(szCurDir, DirEntry.szName);
+                            char *pszFileSource = RTPathJoinA(szCurDir, pDirEntry->szName);
                             if (pszFileSource)
                             {
                                 char *pszFileDest;
-                                vrc = gctlCopyTranslatePath(pszSource, pszFileSource,
-                                                           pszDest, &pszFileDest);
+                                vrc = gctlCopyTranslatePath(pszSource, pszFileSource, pszDest, &pszFileDest);
                                 if (RT_SUCCESS(vrc))
                                 {
@@ -2342 +2337 @@
             }
 
+            RTDirReadExAFree(&pDirEntry, &cbDirEntry);
             RTDirClose(hDir);
         }