From 1913feacb1ad1fb6433d316d81e2e3bee365ec19 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Rafa=C5=82=20Mi=C5=82ecki?= Date: Fri, 30 Nov 2018 18:26:12 +0100 Subject: [PATCH] block: don't duplicate unmounting code in the mount_action() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Use umount_device() helper instead. Signed-off-by: Rafał Miłecki --- block.c | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/block.c b/block.c index e294ba4..46050b4 100644 --- a/block.c +++ b/block.c @@ -1105,27 +1105,17 @@ static int umount_device(char *path) static int mount_action(char *action, char *device, int type) { char path[32]; - char *mount_point; if (!action || !device) return -1; snprintf(path, sizeof(path), "/dev/%s", device); if (!strcmp(action, "remove")) { - int err = 0; - if (type == TYPE_HOTPLUG) blockd_notify(device, NULL, NULL); - mount_point = find_mount_point(path); - if (mount_point) - err = umount2(mount_point, MNT_DETACH); - - if (err) - ULOG_ERR("umount of %s failed (%d) - %m\n", - mount_point, errno); + umount_device(path); - free(mount_point); return 0; } else if (strcmp(action, "add")) { ULOG_ERR("Unkown action %s\n", action); -- 2.30.2