Changeset 99674 in vbox for trunk/src/VBox/VMM/testcase

Timestamp:

May 8, 2023 1:27:47 PM (2 years ago)

Author:

vboxsync

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

157203

Message:

VMM/PDMQueue: Fix PDMQueueAllocEx() to respect the multiple of 32 cBits input parameter requirement (mainly to fix the tstPDMQueue on ARM which uses a different ASMBitFirstSet() implementation which is not as forgiving), and a small fix to the testcase itself to not crash when allocating an item failed, which is counted as an error anyway

File:

• trunk/src/VBox/VMM/testcase/tstPDMQueue.cpp (modified) (1 diff)

trunk/src/VBox/VMM/testcase/tstPDMQueue.cpp

@@ -313 +313 @@
     PTEST1ITEM pMyItem = (PTEST1ITEM)PDMQueueAlloc(pVM, hQueue, pVM);
     RTTEST_CHECK(g_hTest, pMyItem);
-    pMyItem->iSeqNo = 1;
-    RTTEST_CHECK_RC(g_hTest, PDMQueueInsert(pVM, hQueue, pVM, &pMyItem->Core), VINF_SUCCESS);
-
-    PDMR3QueueFlushAll(pVM);
-    RTTEST_CHECK(g_hTest, g_cTest1Callbacks == 1);
+    if (pMyItem)
+    {
+        pMyItem->iSeqNo = 1;
+        RTTEST_CHECK_RC(g_hTest, PDMQueueInsert(pVM, hQueue, pVM, &pMyItem->Core), VINF_SUCCESS);
+
+        PDMR3QueueFlushAll(pVM);
+        RTTEST_CHECK(g_hTest, g_cTest1Callbacks == 1);
+    }
 
     /*