[Libguestfs] [PATCHx7] Misc patches
Jim Meyering
jim at meyering.net
Thu Aug 13 14:59:57 UTC 2009
Richard W.M. Jones wrote:
>>From dcd67e2c4e81bb270ea7f5ed8b6675b114afca60 Mon Sep 17 00:00:00 2001
> From: Richard Jones <rjones at trick.home.annexia.org>
> Date: Thu, 13 Aug 2009 14:41:05 +0100
> Subject: [PATCH 4/7] In rmmountpoint, have to explicitly check for ABS_PATH (*not* NEED_ROOT).
>
> ---
> daemon/mount.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/daemon/mount.c b/daemon/mount.c
> index 4f74c02..d1b26fa 100644
> --- a/daemon/mount.c
> +++ b/daemon/mount.c
> @@ -373,6 +373,9 @@ do_rmmountpoint (const char *path)
> {
> int r;
>
> + /* NEED_ROOT (return -1); - we don't want this test for this call. */
> + ABS_PATH (path, return -1);
> +
> CHROOT_IN;
> r = rmdir (path);
> CHROOT_OUT;
Why? I note that before my commit, 84fc760439e82e6b3616abd0d1f9bd7d7eb01ec0
it had both:
- NEED_ROOT (return -1);
- ABS_PATH (path, return -1);
-
When I switched back from Pathname to String,
shouldn't I have restored both?
More information about the Libguestfs
mailing list