[PATCH 2/2] kvm tools: Cleanup ghost socket files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Clean ghost socket files as part of the regular connection process.

This fixes the bug of 'kvm list' failing with "Failed connecting to instance"
when ghost socket files existed.

Signed-off-by: Sasha Levin <levinsasha928@xxxxxxxxx>
---
 tools/kvm/kvm.c |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/tools/kvm/kvm.c b/tools/kvm/kvm.c
index d953b94..252bd18 100644
--- a/tools/kvm/kvm.c
+++ b/tools/kvm/kvm.c
@@ -208,8 +208,13 @@ int kvm__get_sock_by_instance(const char *name)
 	len = strlen(local.sun_path) + sizeof(local.sun_family);
 
 	r = connect(s, &local, len);
-	if (r < 0)
+	if (r < 0 && errno == ECONNREFUSED) {
+		/* Clean ghost socket file */
+		unlink(sock_file);
+		return -1;
+	} else if (r < 0) {
 		die("Failed connecting to instance");
+	}
 
 	return s;
 }
@@ -232,6 +237,8 @@ int kvm__enumerate_instances(int (*callback)(const char *name, int fd))
 		if (entry.d_type == DT_SOCK) {
 			entry.d_name[strlen(entry.d_name)-5] = 0;
 			sock = kvm__get_sock_by_instance(entry.d_name);
+			if (sock < 0)
+				continue;
 			ret = callback(entry.d_name, sock);
 			close(sock);
 			if (ret < 0)
-- 
1.7.7.1

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux