Changeset 80403 in vbox for trunk/src/VBox/Devices

Timestamp:

Aug 23, 2019 4:03:40 PM (5 years ago)

Author:

vboxsync

Message:

Serial/NamedPipe: don't leak the socket if connect(2) fails.

File:

• trunk/src/VBox/Devices/Serial/DrvNamedPipe.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Serial/DrvNamedPipe.cpp

@@ -1038 +1038 @@
         /* Connect to the local socket. */
         if (connect(s, (struct sockaddr *)&addr, sizeof(addr)) == -1)
+        {
+            close(s);
             return PDMDrvHlpVMSetError(pDrvIns, RTErrConvertFromErrno(errno), RT_SRC_POS,
                                        N_("NamedPipe#%d failed to connect to local socket %s"),
                                        pDrvIns->iInstance, pThis->pszLocation);
+        }
 
         rc = RTSocketFromNative(&pThis->hSock, s);