diff options
| author | Kotresh HR <khiremat@redhat.com> | 2018-05-25 08:55:11 -0400 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-05-28 04:57:22 +0000 |
| commit | 3d38e4e47f129bdb36c3fbbd481dabe4ba4413d6 (patch) | |
| tree | 984b6d3bf3a1554b9e47d964c9f7d89c251f6e34 /api/src/glfs-resolve.c | |
| parent | 7d92af4eac386001b46b8f1a98e8ccb95ca1a225 (diff) | |
libgfapi: Fix lookup on root
Lookup on root was sending "/" as the path.
This will break the basename calculation in
loc_copy and hence lookup on root was failing
if the loc_copy was involved in the stack.
With ctime, a first lookup on root initiates
a metadata self heal because of ctime xattr
not being same on all afr subvolumes. This
results in loc_copy and hence the failure
of lookup.
Fix would be to send path with "." for the root.
fixes: bz#1582516
Change-Id: Iafe4b99f249a4f5034ad34c1d30590de0e35aa0d
Signed-off-by: Kotresh HR <khiremat@redhat.com>
Diffstat (limited to 'api/src/glfs-resolve.c')
| -rw-r--r-- | api/src/glfs-resolve.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/api/src/glfs-resolve.c b/api/src/glfs-resolve.c index 03be7c7eda7..d305c126cb6 100644 --- a/api/src/glfs-resolve.c +++ b/api/src/glfs-resolve.c @@ -241,9 +241,9 @@ glfs_resolve_root (struct glfs *fs, xlator_t *subvol, inode_t *inode, loc.inode = inode_ref (inode); - ret = inode_path (loc.inode, NULL, &path); + ret = inode_path (loc.inode, ".", &path); loc.path = path; - loc.name = ""; + loc.name = "."; /* Having a value in loc.name will help to bypass md-cache check for * nameless lookup. * TODO: Re-visit on nameless lookup and md-cache. |