summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWanlong Gao <gaowanlong@cn.fujitsu.com>2012-07-24 16:47:23 +0800
committerRichard W.M. Jones <rjones@redhat.com>2012-07-24 13:01:08 +0100
commitd8b2c1afee4e23ca864727bebc53a04c490715bc (patch)
tree6fcca62ef31ea93e979e7f1c1b1a86a68b5f326d
parentc03cdcc25ca9b4d2169f5566ea1470758caef812 (diff)
downloadlibguestfs-d8b2c1afee4e23ca864727bebc53a04c490715bc.tar.gz
libguestfs-d8b2c1afee4e23ca864727bebc53a04c490715bc.tar.xz
libguestfs-d8b2c1afee4e23ca864727bebc53a04c490715bc.zip
umount: use Dev_or_Path for the argument type
Use Dev_or_Path instead of String. Remove the RESOLVE_DEVICE since Dev_or_Path will generate REQUIRE_ROOT_OR_RESOLVE_DEVICE instead. RWMJ: Note a change in semantics: this now requires root. However this is OK and still works with mkmountpoint and friends because 'is_root_mounted' works even if something is mounted below the root. Signed-off-by: Wanlong Gao <gaowanlong@cn.fujitsu.com>
-rw-r--r--daemon/mount.c3
-rw-r--r--generator/generator_actions.ml2
2 files changed, 1 insertions, 4 deletions
diff --git a/daemon/mount.c b/daemon/mount.c
index b5b932d8..bd27f94c 100644
--- a/daemon/mount.c
+++ b/daemon/mount.c
@@ -205,9 +205,6 @@ do_umount (const char *pathordevice,
return -1;
}
- if (is_dev)
- RESOLVE_DEVICE (buf, , { free (buf); return -1; });
-
if (!(optargs_bitmask & GUESTFS_UMOUNT_FORCE_BITMASK))
force = 0;
if (!(optargs_bitmask & GUESTFS_UMOUNT_LAZYUNMOUNT_BITMASK))
diff --git a/generator/generator_actions.ml b/generator/generator_actions.ml
index 0adcf293..ecb2e8cd 100644
--- a/generator/generator_actions.ml
+++ b/generator/generator_actions.ml
@@ -2772,7 +2772,7 @@ characters does I<not> work, even if the length is specified." };
{ defaults with
name = "umount";
- style = RErr, [String "pathordevice"], [OBool "force"; OBool "lazyunmount"];
+ style = RErr, [Dev_or_Path "pathordevice"], [OBool "force"; OBool "lazyunmount"];
proc_nr = Some 45;
fish_alias = ["unmount"];
once_had_no_optargs = true;