[LEDE-DEV] [PATCH mountd] mount: change mount_dev_del() argument to struct mount *

Rafał Miłecki zajec5 at gmail.com
Fri Feb 9 07:21:49 PST 2018


From: Rafał Miłecki <rafal at milecki.pl>

Caller of mount_dev_del() already has struct mount * so there is no
point in passing matchin device and then looking for struct mount *
again.

Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
---
 mount.c | 28 ++++++++++++----------------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/mount.c b/mount.c
index f995745..646cc30 100644
--- a/mount.c
+++ b/mount.c
@@ -567,23 +567,19 @@ static void mount_dev_add(char *dev)
 	}
 }
 
-static void mount_dev_del(char *dev)
+static void mount_dev_del(struct mount *mount)
 {
-	struct mount *mount = mount_find(0, dev);
 	char tmp[256];
-	if(mount)
-	{
-		if(mount->mounted)
-		{
-			snprintf(tmp, 256, "%s%s", "/tmp/run/mountd/", mount->name);
-			log_printf("%s has dissappeared ... unmounting\n", tmp);
-			snprintf(tmp, 256, "%s%s", "/tmp/run/mountd/", mount->dev);
-			system_printf("/bin/umount %s", tmp);
-			rmdir(tmp);
-			snprintf(tmp, 64, "%s%s", uci_path, mount->name);
-			unlink(tmp);
-			mount_dump_uci_state();
-		}
+
+	if (mount->mounted) {
+		snprintf(tmp, 256, "%s%s", "/tmp/run/mountd/", mount->name);
+		log_printf("%s has dissappeared ... unmounting\n", tmp);
+		snprintf(tmp, 256, "%s%s", "/tmp/run/mountd/", mount->dev);
+		system_printf("/bin/umount %s", tmp);
+		rmdir(tmp);
+		snprintf(tmp, 64, "%s%s", uci_path, mount->name);
+		unlink(tmp);
+		mount_dump_uci_state();
 	}
 }
 
@@ -748,7 +744,7 @@ static void mount_enum_drives(void)
 		}
 		if(!check_block(q->dev)||del)
 		{
-			mount_dev_del(q->dev);
+			mount_dev_del(q);
 			p->prev->next = p->next;
 			p->next->prev = p->prev;
 			p = p->next;
-- 
2.11.0




More information about the Lede-dev mailing list