Skip to content

Commit 68f6565

Browse files
committed
CDRIVER-756: Simplify mongoc_socket_close()
1 parent 5da9035 commit 68f6565

File tree

1 file changed

+14
-15
lines changed

1 file changed

+14
-15
lines changed

src/mongoc/mongoc-socket.c

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -523,27 +523,26 @@ mongoc_socket_close (mongoc_socket_t *sock) /* IN */
523523
#ifdef _WIN32
524524
if (sock->sd != INVALID_SOCKET) {
525525
shutdown (sock->sd, SD_BOTH);
526-
ret = closesocket (sock->sd);
526+
if (0 == closesocket (sock->sd)) {
527+
sock->sd = INVALID_SOCKET;
528+
} else {
529+
_mongoc_socket_capture_errno (sock);
530+
RETURN(-1);
531+
}
527532
}
533+
RETURN(0);
528534
#else
529535
if (sock->sd != -1) {
530536
shutdown (sock->sd, SHUT_RDWR);
531-
ret = close (sock->sd);
537+
if (0 == close (sock->sd)) {
538+
sock->sd = -1;
539+
} else {
540+
_mongoc_socket_capture_errno (sock);
541+
RETURN(-1);
542+
}
532543
}
544+
RETURN(0);
533545
#endif
534-
535-
_mongoc_socket_capture_errno (sock);
536-
537-
if (ret == 0) {
538-
#ifdef _WIN32
539-
sock->sd = INVALID_SOCKET;
540-
#else
541-
sock->sd = -1;
542-
#endif
543-
RETURN (0);
544-
}
545-
546-
RETURN (-1);
547546
}
548547

549548

0 commit comments

Comments
 (0)