Changeset 10839

Timestamp:

07/23/08 21:48:51 (3 months ago)

Author:

vboxsync

Message:

RTSemEventDestroy: Don't bitch on NIL_RTSEMEVENT.

Files:

• trunk/src/VBox/Runtime/r3/linux/semevent-linux.cpp (modified) (1 diff)
• trunk/src/VBox/Runtime/r3/os2/sems-os2.cpp (modified) (1 diff)
• trunk/src/VBox/Runtime/r3/posix/semevent-posix.cpp (modified) (1 diff)
• trunk/src/VBox/Runtime/r3/win/sems-win.cpp (modified) (2 diffs)

trunk/src/VBox/Runtime/r3/linux/semevent-linux.cpp

@@ -111 +111 @@
-     * Validate input.
-     */
+    if (EventSem == NIL_RTSEMEVENT)     /* don't bitch */
+        return VERR_INVALID_HANDLE;
-    struct RTSEMEVENTINTERNAL *pThis = EventSem;
-Return(VALID_PTR(pThis) && pThis->iMagic == RTSEMEVENT_MAGIC,
-            
+PtrReturn(pThis, VERR_INVALID_HANDLE);
+AssertReturn(pThis->iMagic == RTSEMEVENT_MAGIC,
-
-    /*

trunk/src/VBox/Runtime/r3/os2/sems-os2.cpp

@@ -66 +66 @@
-RTDECL(int)   RTSemEventDestroy(RTSEMEVENT EventSem)
-{
+    if (EventSem == NIL_RTSEMEVENT)     /* don't bitch */
+        return VERR_INVALID_HANDLE;
+
-    /*
-     * Close semaphore handle.

trunk/src/VBox/Runtime/r3/posix/semevent-posix.cpp

@@ -167 +167 @@
-     * Validate handle.
-     */
+    if (EventSem == NIL_RTSEMEVENT)     /* don't bitch */
+        return VERR_INVALID_HANDLE;
-    if (!rtsemEventValid(EventSem))
-    {

trunk/src/VBox/Runtime/r3/win/sems-win.cpp

@@ -61 +61 @@
-    {
-        *pEventSem = (RTSEMEVENT)(void *)hev;
+        Assert(*pEventSem != NIL_RTSEMEVENT);
-        return VINF_SUCCESS;
-    }
@@ -69 +70 @@
-RTDECL(int)   RTSemEventDestroy(RTSEMEVENT EventSem)
-{
+    if (EventSem == NIL_RTSEMEVENT)     /* don't bitch */
+        return VERR_INVALID_HANDLE;
+
-    /*
-     * Close semaphore handle.