External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-07-11 04:15:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

qemuMonitorOpenUnix: Don't overwrite 'ret' needlessly

Browse Source 
Directly check the return value of 'connect'. Unfortunately we can't
remove it as we have to undo auto-closing of the socket on success.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Jonathon Jongsma <jjongsma@redhat.com>
This commit is contained in:
Peter Krempa 2022-08-02 14:41:31 +02:00
parent 31699d3914
commit 469f2ab31c
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/qemu/qemu_monitor.c
View File

@ -245,8 +245,7 @@ qemuMonitorOpenUnix(const char *monitor)
return -1;
}
ret = connect(monfd, (struct sockaddr *) &addr, sizeof(addr));
if (ret < 0) {
if (connect(monfd, (struct sockaddr *) &addr, sizeof(addr)) < 0) {
virReportSystemError(errno, "%s",
_("failed to connect to monitor socket"));
return -1;