Changeset 90813 in vbox for trunk/src

Timestamp:

Aug 23, 2021 8:09:42 PM (3 years ago)

Author:

vboxsync

Message:

Storage/VDDumpImage: Don't crash if pDisk->pInterfaceError is NULL.

File:

• trunk/src/VBox/Storage/VD.cpp (modified) (1 diff)

trunk/src/VBox/Storage/VD.cpp

@@ -9413 +9413 @@
     AssertMsg(pDisk->u32Signature == VDISK_SIGNATURE, ("u32Signature=%08x\n", pDisk->u32Signature));
 
-    if (!pDisk->pInterfaceError || !RT_VALID_PTR(pDisk->pInterfaceError->pfnMessage)) /** @todo r=bird: first test is boinkers. */
+    AssertPtrReturnVoid(pDisk->pInterfaceError);
+    if (!RT_VALID_PTR(pDisk->pInterfaceError->pfnMessage))
         pDisk->pInterfaceError->pfnMessage = vdLogMessage;