[Libguestfs] [PATCH 1/5] fuse: Fix symlink creation (RHBZ#538069).

Matthew Booth mbooth at redhat.com
Tue Nov 17 17:56:53 UTC 2009


On 17/11/09 17:06, Richard W.M. Jones wrote:
>> From 93514b1fdb56a0cb6b85b79aff9f08fbd71a9560 Mon Sep 17 00:00:00 2001
> From: Richard Jones<rjones at redhat.com>
> Date: Tue, 17 Nov 2009 16:59:52 +0000
> Subject: [PATCH 1/5] fuse: Fix symlink creation (RHBZ#538069).
>
> The parameters were swapped, preventing symlinks from being created.
> Furthermore, we need to invalidate the cache for both parameters.
> ---
>   fuse/guestmount.c |    3 ++-
>   1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/fuse/guestmount.c b/fuse/guestmount.c
> index 739d8cb..8e081a6 100644
> --- a/fuse/guestmount.c
> +++ b/fuse/guestmount.c
> @@ -425,9 +425,10 @@ fg_symlink (const char *from, const char *to)
>
>     if (read_only) return -EROFS;
>
> +  dir_cache_invalidate (from);

As discussed on IRC, this isn't necessary (and could be confusing).

>     dir_cache_invalidate (to);
>
> -  r = guestfs_ln_s (g, to, from);
> +  r = guestfs_ln_s (g, from, to);
>     if (r == -1)
>       return error ();
>
> -- 1.6.5.2

Apart from that, ACK.

Matt
-- 
Matthew Booth, RHCA, RHCSS
Red Hat Engineering, Virtualisation Team

M:       +44 (0)7977 267231
GPG ID:  D33C3490
GPG FPR: 3733 612D 2D05 5458 8A8A 1600 3441 EA19 D33C 3490




More information about the Libguestfs mailing list