summaryrefslogtreecommitdiffstats
path: root/libglusterfs
diff options
context:
space:
mode:
authorAmar Tumballi <amar@gluster.com>2011-10-02 08:46:46 +0530
committerVijay Bellur <vijay@gluster.com>2011-10-02 22:08:02 -0700
commitb6eee04da4a699c7cd850bf2121825cc67f14707 (patch)
tree58ec2747855ba5c37330414b4423fd33e206b80c /libglusterfs
parent0db88567b916dc0dbfb0dfe2ed8a47f8d50fb317 (diff)
core: made changes to return value of __is_root_gfid()
now returns 'true(1)' is gfid is root, 'false(0)' if not. earlier it was the inverse, which was bit confusing Change-Id: Id103f444ace048cbb0fccdc72c6646da06631584 BUG: 3518 Reviewed-on: http://review.gluster.com/549 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
Diffstat (limited to 'libglusterfs')
-rw-r--r--libglusterfs/src/compat.c4
-rw-r--r--libglusterfs/src/inode.c11
-rw-r--r--libglusterfs/src/inode.h3
3 files changed, 10 insertions, 8 deletions
diff --git a/libglusterfs/src/compat.c b/libglusterfs/src/compat.c
index e6c7f41e1a5..20e165a7a1b 100644
--- a/libglusterfs/src/compat.c
+++ b/libglusterfs/src/compat.c
@@ -121,7 +121,7 @@ make_export_path (const char *real_path, char **path)
ret = solaris_getxattr ("/", GFID_XATTR_KEY, gfid, 16);
/* Return value of getxattr */
if (ret == 16) {
- if (!__is_root_gfid (gfid)){
+ if (__is_root_gfid (gfid)){
strcat (export_path, "/");
ret = 0;
goto done;
@@ -135,7 +135,7 @@ make_export_path (const char *real_path, char **path)
strcat (export_path, dup);
ret = solaris_getxattr (export_path, GFID_XATTR_KEY, gfid, 16);
if (ret == 16) {
- if (!__is_root_gfid (gfid)) {
+ if (__is_root_gfid (gfid)) {
ret = 0;
goto done;
}
diff --git a/libglusterfs/src/inode.c b/libglusterfs/src/inode.c
index 117b977b924..3239878b35d 100644
--- a/libglusterfs/src/inode.c
+++ b/libglusterfs/src/inode.c
@@ -653,18 +653,19 @@ inode_grep (inode_table_t *table, inode_t *parent, const char *name)
return inode;
}
-int
+/* return 1 if gfid is of root, 0 if not */
+gf_boolean_t
__is_root_gfid (uuid_t gfid)
{
uuid_t root;
- int ret;
memset (root, 0, 16);
root[15] = 1;
- ret = uuid_compare (gfid, root);
+ if (uuid_compare (gfid, root) == 0)
+ return _gf_true;
- return ret;
+ return _gf_false;
}
@@ -680,7 +681,7 @@ __inode_find (inode_table_t *table, uuid_t gfid)
goto out;
}
- if (__is_root_gfid (gfid) == 0)
+ if (__is_root_gfid (gfid))
return table->root;
hash = hash_gfid (gfid, 65536);
diff --git a/libglusterfs/src/inode.h b/libglusterfs/src/inode.h
index b1d9b3742ea..a632ff5e965 100644
--- a/libglusterfs/src/inode.h
+++ b/libglusterfs/src/inode.h
@@ -182,6 +182,7 @@ int
inode_ctx_del2 (inode_t *inode, xlator_t *xlator, uint64_t *value1,
uint64_t *value2);
-int
+gf_boolean_t
__is_root_gfid (uuid_t gfid);
+
#endif /* _INODE_H */