[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 11/27] vhost-user-scsi: simplify unix path cleanup
From: |
Marc-André Lureau |
Subject: |
[Qemu-devel] [PATCH 11/27] vhost-user-scsi: simplify unix path cleanup |
Date: |
Wed, 23 Aug 2017 18:19:48 +0200 |
Always remove the unix path when leaving the program (instead of when
freeing scsi_dev). Note that unix_sock_new() also unlink() exisiting
path before creating the socket.
Signed-off-by: Marc-André Lureau <address@hidden>
---
contrib/vhost-user-scsi/vhost-user-scsi.c | 22 +++++-----------------
1 file changed, 5 insertions(+), 17 deletions(-)
diff --git a/contrib/vhost-user-scsi/vhost-user-scsi.c
b/contrib/vhost-user-scsi/vhost-user-scsi.c
index bb10848cd2..624efc24ca 100644
--- a/contrib/vhost-user-scsi/vhost-user-scsi.c
+++ b/contrib/vhost-user-scsi/vhost-user-scsi.c
@@ -636,24 +636,9 @@ fail:
static void vdev_scsi_free(vhost_scsi_dev_t *vdev_scsi)
{
- if (!vdev_scsi) {
- return;
- }
-
if (vdev_scsi->server_sock >= 0) {
- struct sockaddr_storage ss;
- socklen_t sslen = sizeof(ss);
-
- if (getsockname(vdev_scsi->server_sock, (struct sockaddr *)&ss,
- &sslen) == 0) {
- struct sockaddr_un *su = (struct sockaddr_un *)&ss;
- (void)unlink(su->sun_path);
- }
-
- (void)close(vdev_scsi->server_sock);
- vdev_scsi->server_sock = -1;
+ close(vdev_scsi->server_sock);
}
-
g_main_loop_unref(vdev_scsi->loop);
g_tree_destroy(vdev_scsi->fdmap);
g_free(vdev_scsi);
@@ -764,7 +749,10 @@ int main(int argc, char **argv)
}
out:
- vdev_scsi_free(vdev_scsi);
+ if (vdev_scsi) {
+ vdev_scsi_free(vdev_scsi);
+ unlink(unix_fn);
+ }
g_free(unix_fn);
g_free(iscsi_uri);
--
2.14.1.146.gd35faa819
- [Qemu-devel] [PATCH 02/27] libvhost-user: drop dependency on glib, (continued)
- [Qemu-devel] [PATCH 02/27] libvhost-user: drop dependency on glib, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 04/27] vhost-user-scsi: use g_strdup(), Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 05/27] vhost-user-scsi: connect unix socket before allocating, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 03/27] libvhost-user: improve vu_queue_pop() doc, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 07/27] vhost-user-scsi: use glib allocation, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 06/27] vhost-user-scsi: code style fixes, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 08/27] vhost-user-scsi: glib calls that allocate don't return NULL, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 09/27] vhost-user-scsi: also free the gtree, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 11/27] vhost-user-scsi: simplify unix path cleanup,
Marc-André Lureau <=
- [Qemu-devel] [PATCH 10/27] vhost-user-scsi: remove vdev_scsi_find_by_vu(), Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 12/27] vhost-user-scsi: use NULL pointer, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 13/27] vhost-user-scsi: use glib watch directly, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 14/27] vhost-user-scsi: assert() in iscsi_add_lun(), Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 15/27] vhost-user-scsi: remove vdev_scsi_add_iscsi_lun(), Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 17/27] vhost-user-scsi: remove unimplemented functions, Marc-André Lureau, 2017/08/23
- [Qemu-devel] [PATCH 16/27] vhost-user-scsi: remove VUS_MAX_LUNS, Marc-André Lureau, 2017/08/23