Changeset 67151 in vbox for trunk/src/VBox/Runtime

Timestamp:

May 31, 2017 7:12:18 AM (8 years ago)

Author:

vboxsync

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

115827

Message:

Runtime/r3/socket: when checking for select() results it is possible that pThis->hNative became invalid in the meantimebecause the socket was closed, see rtSocketCloseIt() where we invalid pThis->hNative first and then close the socket. We could also remember hNative for the correct test after select() but I don't think we really care about the RTSOCKET_EVT_READ result on closed sockets.

File:

• trunk/src/VBox/Runtime/r3/socket.cpp (modified) (1 diff)

trunk/src/VBox/Runtime/r3/socket.cpp

@@ -1545 +1545 @@
     if (rc > 0)
     {
-        if (FD_ISSET(pThis->hNative, &fdsetR))
-            *pfEvents |= RTSOCKET_EVT_READ;
-        if (FD_ISSET(pThis->hNative, &fdsetW))
-            *pfEvents |= RTSOCKET_EVT_WRITE;
-        if (FD_ISSET(pThis->hNative, &fdsetE))
-            *pfEvents |= RTSOCKET_EVT_ERROR;
+        if (pThis->hNative != NIL_RTSOCKETNATIVE)
+        {
+            if (FD_ISSET(pThis->hNative, &fdsetR))
+                *pfEvents |= RTSOCKET_EVT_READ;
+            if (FD_ISSET(pThis->hNative, &fdsetW))
+                *pfEvents |= RTSOCKET_EVT_WRITE;
+            if (FD_ISSET(pThis->hNative, &fdsetE))
+                *pfEvents |= RTSOCKET_EVT_ERROR;
+        }
 
         rc = VINF_SUCCESS;