Changeset 83084 in vbox for trunk/src/VBox/Runtime/common/dbg

Timestamp:

Feb 15, 2020 3:16:11 PM (5 years ago)

Author:

vboxsync

Message:

IPRT,DBGPlugInDarwin: Added flags for indicating that the LINKEDIT segment should be loaded in Mach-O images and debug files.

Location:

trunk/src/VBox/Runtime/common/dbg

Files:

• dbgmod.cpp (modified) (12 diffs)
• dbgmoddeferred.cpp (modified) (3 diffs)
• dbgmoddwarf.cpp (modified) (1 diff)
• dbgmodldr.cpp (modified) (1 diff)

trunk/src/VBox/Runtime/common/dbg/dbgmod.cpp

@@ -329 +329 @@
     AssertPtrReturn(pszName, VERR_INVALID_POINTER);
     AssertReturn(*pszName, VERR_INVALID_PARAMETER);
-    AssertReturn(fFlags == 0 || fFlags == RTDBGMOD_F_NOT_DEFERRED, VERR_INVALID_PARAMETER);
+    AssertReturn(fFlags == 0 || fFlags == RTDBGMOD_F_NOT_DEFERRED, VERR_INVALID_FLAGS);
 
     int rc = rtDbgModLazyInit();
@@ -842 +842 @@
                         pDbgMod->pvImgPriv = NULL;
                         /** @todo need to specify some arch stuff here. */
-                        rc = pImg->pVt->pfnTryOpen(pDbgMod, enmArch);
+                        rc = pImg->pVt->pfnTryOpen(pDbgMod, enmArch, 0 /*fLdrFlags*/);
                         if (RT_SUCCESS(rc))
                         {
@@ -980 +980 @@
             pDbgMod->pImgVt    = pImg->pVt;
             pDbgMod->pvImgPriv = NULL;
-            int rc2 = pImg->pVt->pfnTryOpen(pDbgMod, RTLDRARCH_WHATEVER);
+            int rc2 = pImg->pVt->pfnTryOpen(pDbgMod, RTLDRARCH_WHATEVER, 0 /*fLdrFlags*/);
             if (RT_SUCCESS(rc2))
             {
@@ -1159 +1159 @@
                     {
                         PRTDBGMODDEFERRED pDeferred;
-                        rc = rtDbgModDeferredCreate(pDbgMod, rtDbgModFromPeImageDeferredCallback, cbImage, hDbgCfg, 0,
-                                                    &pDeferred);
+                        rc = rtDbgModDeferredCreate(pDbgMod, rtDbgModFromPeImageDeferredCallback, cbImage, hDbgCfg,
+                                                    0 /*cbDeferred*/, 0 /*fFlags*/, &pDeferred);
                         if (RT_SUCCESS(rc))
                             pDeferred->u.PeImage.uTimestamp = uTimestamp;
@@ -1217 +1217 @@
     /** For use more internal use in file locator callbacks. */
     bool                fOpenImage;
+    /** RTDBGMOD_F_XXX. */
+    uint32_t            fFlags;
 } RTDBGMODMACHOARGS;
 /** Pointer to a const segment package. */
@@ -1259 +1261 @@
             pDbgMod->pImgVt    = pImg->pVt;
             pDbgMod->pvImgPriv = NULL;
-            int rc2 = pImg->pVt->pfnTryOpen(pDbgMod, pArgs->enmArch);
+            int rc2 = pImg->pVt->pfnTryOpen(pDbgMod, pArgs->enmArch,
+                                            pArgs->fFlags & RTDBGMOD_F_MACHO_LOAD_LINKEDIT ? RTLDR_O_MACHO_LOAD_LINKEDIT : 0);
             if (RT_SUCCESS(rc2))
             {
@@ -1350 +1353 @@
 
 static int rtDbgModFromMachOImageWorker(PRTDBGMODINT pDbgMod, RTLDRARCH enmArch, uint32_t cbImage,
-                                        uint32_t cSegs, PCRTDBGSEGMENT paSegs, PCRTUUID pUuid, RTDBGCFG hDbgCfg)
+                                        uint32_t cSegs, PCRTDBGSEGMENT paSegs, PCRTUUID pUuid, RTDBGCFG hDbgCfg, uint32_t fFlags)
 {
     RT_NOREF_PV(cbImage); RT_NOREF_PV(cSegs); RT_NOREF_PV(paSegs);
@@ -1358 +1361 @@
     Args.pUuid      = pUuid && RTUuidIsNull(pUuid) ? pUuid : NULL;
     Args.fOpenImage = false;
+    Args.fFlags     = fFlags;
 
     /*
@@ -1382 +1386 @@
     return rtDbgModFromMachOImageWorker(pDbgMod, pDeferred->u.MachO.enmArch, pDeferred->cbImage,
                                         pDeferred->u.MachO.cSegs, pDeferred->u.MachO.aSegs,
-                                        &pDeferred->u.MachO.Uuid, pDeferred->hDbgCfg);
+                                        &pDeferred->u.MachO.Uuid, pDeferred->hDbgCfg, pDeferred->fFlags);
 }
 
@@ -1408 +1412 @@
     AssertReturn(cbImage || cSegs, VERR_INVALID_PARAMETER);
     AssertPtrNullReturn(pUuid, VERR_INVALID_POINTER);
-    AssertReturn(!(fFlags & ~(RTDBGMOD_F_NOT_DEFERRED)), VERR_INVALID_PARAMETER);
+    AssertReturn(!(fFlags & ~RTDBGMOD_F_VALID_MASK), VERR_INVALID_FLAGS);
 
     int rc = rtDbgModLazyInit();
@@ -1448 +1452 @@
                     || (!cbImage && !cSegs)
                     || (fFlags & RTDBGMOD_F_NOT_DEFERRED) )
-                    rc = rtDbgModFromMachOImageWorker(pDbgMod, enmArch, cbImage, cSegs, paSegs, pUuid, hDbgCfg);
+                    rc = rtDbgModFromMachOImageWorker(pDbgMod, enmArch, cbImage, cSegs, paSegs, pUuid, hDbgCfg, fFlags);
                 else
                 {
@@ -1457 +1461 @@
                     rc = rtDbgModDeferredCreate(pDbgMod, rtDbgModFromMachOImageDeferredCallback, cbImage, hDbgCfg,
                                                 RT_UOFFSETOF_DYN(RTDBGMODDEFERRED, u.MachO.aSegs[cSegs]),
-                                                &pDeferred);
+                                                0 /*fFlags*/, &pDeferred);
                     if (RT_SUCCESS(rc))
                     {

trunk/src/VBox/Runtime/common/dbg/dbgmoddeferred.cpp

@@ -636 +636 @@
 
 /** @interface_method_impl{RTDBGMODVTIMG,pfnTryOpen} */
-static DECLCALLBACK(int) rtDbgModDeferredImg_TryOpen(PRTDBGMODINT pMod, RTLDRARCH enmArch)
-{
-    NOREF(enmArch);
+static DECLCALLBACK(int) rtDbgModDeferredImg_TryOpen(PRTDBGMODINT pMod, RTLDRARCH enmArch, uint32_t fLdrFlags)
+{
+    RT_NOREF(enmArch, fLdrFlags);
     return rtDbgModDeferredDoIt(pMod, true /*fForceRetry*/);
 }
@@ -681 +681 @@
  * @param   cbDeferred          The size of the deferred instance data, 0 if the
  *                              default structure is good enough.
+ * @param   fFlags              RTDBGMOD_F_XXX.
  * @param   ppDeferred          Where to return the instance data. Can be NULL.
  */
 DECLHIDDEN(int) rtDbgModDeferredCreate(PRTDBGMODINT pDbgMod, PFNRTDBGMODDEFERRED pfnDeferred, RTUINTPTR cbImage,
-                                       RTDBGCFG hDbgCfg, size_t cbDeferred, PRTDBGMODDEFERRED *ppDeferred)
+                                       RTDBGCFG hDbgCfg, size_t cbDeferred, uint32_t fFlags, PRTDBGMODDEFERRED *ppDeferred)
 {
     AssertReturn(!pDbgMod->pDbgVt, VERR_DBG_MOD_IPE);
@@ -701 +702 @@
     pDeferred->hDbgCfg     = hDbgCfg;
     pDeferred->pfnDeferred = pfnDeferred;
+    pDeferred->fFlags      = fFlags;
 
     pDbgMod->pDbgVt             = &g_rtDbgModVtDbgDeferred;

trunk/src/VBox/Runtime/common/dbg/dbgmoddwarf.cpp

@@ -6077 +6077 @@
             pDbgInfoMod->pszName = pDbgMod->pszName;
             pDbgInfoMod->pImgVt  = &g_rtDbgModVtImgLdr;
-            rc = pDbgInfoMod->pImgVt->pfnTryOpen(pDbgInfoMod, enmArch);
+            rc = pDbgInfoMod->pImgVt->pfnTryOpen(pDbgInfoMod, enmArch, 0 /*fLdrFlags*/);
             if (RT_SUCCESS(rc))
             {

trunk/src/VBox/Runtime/common/dbg/dbgmodldr.cpp

@@ -216 +216 @@
 
 /** @interface_method_impl{RTDBGMODVTIMG,pfnTryOpen} */
-static DECLCALLBACK(int) rtDbgModLdr_TryOpen(PRTDBGMODINT pMod, RTLDRARCH enmArch)
+static DECLCALLBACK(int) rtDbgModLdr_TryOpen(PRTDBGMODINT pMod, RTLDRARCH enmArch, uint32_t fLdrFlags)
 {
     RTLDRMOD hLdrMod;
-    int rc = RTLdrOpen(pMod->pszImgFile, RTLDR_O_FOR_DEBUG, enmArch, &hLdrMod);
+    int rc = RTLdrOpen(pMod->pszImgFile, RTLDR_O_FOR_DEBUG | fLdrFlags, enmArch, &hLdrMod);
     if (RT_SUCCESS(rc))
     {