Changeset 33043 in vbox for trunk/src/VBox/Runtime/r0drv/linux

Timestamp:

Oct 11, 2010 3:56:04 PM (14 years ago)

Author:

vboxsync

Message:

nt,linux: Added missing flag validation in the RTSemEventMultiWaitEx* implementations.

Location:

trunk/src/VBox/Runtime/r0drv/linux

Files:

• semevent-r0drv-linux.c (modified) (1 diff)
• semeventmulti-r0drv-linux.c (modified) (3 diffs)

trunk/src/VBox/Runtime/r0drv/linux/semevent-r0drv-linux.c

@@ -178 +178 @@
     AssertPtrReturn(pThis, VERR_INVALID_PARAMETER);
     AssertMsgReturn(pThis->u32Magic == RTSEMEVENT_MAGIC, ("%p u32Magic=%RX32\n", pThis, pThis->u32Magic), VERR_INVALID_PARAMETER);
+    AssertReturn(RTSEMWAIT_FLAGS_ARE_VALID(fFlags), VERR_INVALID_PARAMETER);
     rtR0SemEventLnxRetain(pThis);
 

trunk/src/VBox/Runtime/r0drv/linux/semeventmulti-r0drv-linux.c

@@ -60 +60 @@
 /** @}  */
 
+
 /*******************************************************************************
 *   Structures and Typedefs                                                    *
@@ -71 +72 @@
     uint32_t volatile   u32Magic;
     /** The object state bit and generation counter.
-     * The generation counter is incremented every time the object is   */
+     * The generation counter is incremented every time the object is
+     * signalled. */
     uint32_t volatile   fStateAndGen;
     /** Reference counter. */
@@ -241 +243 @@
     AssertPtrReturn(pThis, VERR_INVALID_PARAMETER);
     AssertMsgReturn(pThis->u32Magic == RTSEMEVENTMULTI_MAGIC, ("%p u32Magic=%RX32\n", pThis, pThis->u32Magic), VERR_INVALID_PARAMETER);
+    AssertReturn(RTSEMWAIT_FLAGS_ARE_VALID(fFlags), VERR_INVALID_PARAMETER);
     rtR0SemEventMultiLnxRetain(pThis);