Changeset 104605 in vbox for trunk/src/VBox

Timestamp:

May 13, 2024 4:02:45 PM (7 months ago)

Author:

vboxsync

Message:

GuestHost/DragAndDrop: pcszBaseOld not being NULL is already checked before, act on the status code returned by dndTransferListRootEntry(), bugref:3409

Location:

trunk/src/VBox/GuestHost/DragAndDrop

Files:

• DnDPath.cpp (modified) (1 diff)
• DnDTransferList.cpp (modified) (1 diff)

trunk/src/VBox/GuestHost/DragAndDrop/DnDPath.cpp

@@ -183 +183 @@
     /* Do we need to see if the given path is part of the old base? */
     size_t idxBase;
-    if (   pcszBaseOld
-        && RTPathStartsWith(pcszPath, pcszBaseOld))
-    {
+    if (RTPathStartsWith(pcszPath, pcszBaseOld))
         idxBase = strlen(pcszBaseOld);
-    }
     else
         idxBase = 0;

trunk/src/VBox/GuestHost/DragAndDrop/DnDTransferList.cpp

@@ -543 +543 @@
             {
                 rc = dndTransferListRootEntryAdd(pList, pszPathAbs);
+                if (RT_FAILURE(rc))
+                    break;
             }
         }