Changeset 49789 in vbox for trunk/src/VBox/VMM/VMMR3

Timestamp:

Dec 5, 2013 12:51:16 PM (11 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

91109

Message:

VMM: don't ignore the error code of RTAvl*DoWithAll() if the callback function actually returns codes other than VINF_SUCCESS, in particular not when saving the PATM state

File:

• trunk/src/VBox/VMM/VMMR3/PATMSSM.cpp (modified) (2 diffs)

trunk/src/VBox/VMM/VMMR3/PATMSSM.cpp

@@ -720 +720 @@
     AssertMsg(nrFixupRecs == pPatch->patch.nrFixups, ("Fixup inconsistency! counted %d vs %d\n", nrFixupRecs, pPatch->patch.nrFixups));
 #endif
-    RTAvlPVDoWithAll(&pPatch->patch.FixupTree, true, patmSaveFixupRecords, pVM);
+    rc = RTAvlPVDoWithAll(&pPatch->patch.FixupTree, true, patmSaveFixupRecords, pVM);
+    AssertRCReturn(rc, rc);
 
 #ifdef VBOX_STRICT
@@ -729 +730 @@
 
     RTAvlU32DoWithAll(&pPatch->patch.Patch2GuestAddrTree, true, patmSaveP2GLookupRecords, pVM);
+    AssertRCReturn(rc, rc);
+
     return VINF_SUCCESS;
 }