On 3/1/23 12:08, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau <marcandre.lur...@redhat.com>
Whether it is SPICE, VNC, D-Bus, or the socket chardev, they all
actually expect a socket kind or will fail in different ways at runtime.
Throw an error early if the given 'add_client' fd is not a socket, and
close it to avoid leaks.
This allows to replace the close() call with a more correct & portable
closesocket() version.
(this will allow importing sockets on Windows with a specialized command
in the following patch, while keeping the remaining monitor associated
sockets/add_client code & usage untouched)
Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
---
monitor/qmp-cmds.c | 19 +++++++++++++------
1 file changed, 13 insertions(+), 6 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
