summaryrefslogtreecommitdiffstats
path: root/util
diff options
context:
space:
mode:
authorBin Meng2022-08-02 09:51:57 +0200
committerDaniel P. Berrangé2022-08-05 17:18:15 +0200
commite3fdb13e8851be570db41a50589ce82d11d61c12 (patch)
tree1b237046dd6aafbdadcccfac68e5f67abcb768d6 /util
parenttarget/arm: display deprecation status in '-cpu help' (diff)
downloadqemu-e3fdb13e8851be570db41a50589ce82d11d61c12.tar.gz
qemu-e3fdb13e8851be570db41a50589ce82d11d61c12.tar.xz
qemu-e3fdb13e8851be570db41a50589ce82d11d61c12.zip
util/qemu-sockets: Replace the call to close a socket with closesocket()
close() is a *nix function. It works on any file descriptor, and sockets in *nix are an example of a file descriptor. closesocket() is a Windows-specific function, which works only specifically with sockets. Sockets on Windows do not use *nix-style file descriptors, and socket() returns a handle to a kernel object instead, so it must be closed with closesocket(). In QEMU there is already a logic to handle such platform difference in os-posix.h and os-win32.h, that: * closesocket maps to close on POSIX * closesocket maps to a wrapper that calls the real closesocket() on Windows Replace the call to close a socket with closesocket() instead. Signed-off-by: Bin Meng <bin.meng@windriver.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Diffstat (limited to 'util')
-rw-r--r--util/qemu-sockets.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
index 13b5b197f9..0e2298278f 100644
--- a/util/qemu-sockets.c
+++ b/util/qemu-sockets.c
@@ -487,7 +487,7 @@ int inet_connect_saddr(InetSocketAddress *saddr, Error **errp)
if (ret < 0) {
error_setg_errno(errp, errno, "Unable to set KEEPALIVE");
- close(sock);
+ closesocket(sock);
return -1;
}
}
@@ -1050,7 +1050,7 @@ static int unix_connect_saddr(UnixSocketAddress *saddr, Error **errp)
return sock;
err:
- close(sock);
+ closesocket(sock);
return -1;
}