diff options
| author | Amar Tumballi <amar@gluster.com> | 2011-10-02 08:46:46 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2011-10-02 22:08:02 -0700 |
| commit | b6eee04da4a699c7cd850bf2121825cc67f14707 (patch) | |
| tree | 58ec2747855ba5c37330414b4423fd33e206b80c /xlators | |
| parent | 0db88567b916dc0dbfb0dfe2ed8a47f8d50fb317 (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 'xlators')
| -rw-r--r-- | xlators/cluster/dht/src/dht-common.c | 2 | ||||
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-utils.c | 2 | ||||
| -rw-r--r-- | xlators/storage/posix/src/posix.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index 6cf1d0c85a7..63e505c8c91 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -1829,7 +1829,7 @@ dht_setxattr (call_frame_t *frame, xlator_t *this, tmp = dict_get (xattr, "decommission-brick"); if (tmp) { /* This operation should happen only on '/' */ - if (__is_root_gfid (loc->inode->gfid) != 0) { + if (!__is_root_gfid (loc->inode->gfid)) { op_errno = ENOTSUP; goto err; } diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c index 9ecd78dfaea..0b399da096d 100644 --- a/xlators/mgmt/glusterd/src/glusterd-utils.c +++ b/xlators/mgmt/glusterd/src/glusterd-utils.c @@ -3644,7 +3644,7 @@ check_xattr: other than that of root '/' */ ret = sys_lgetxattr (path, "trusted.gfid", gfid, 16); if (ret == 16) { - if (__is_root_gfid (gfid) != 0) { + if (!__is_root_gfid (gfid)) { gf_log (THIS->name, GF_LOG_WARNING, "%s: gfid (%s) is not that of glusterfs '/' ", path, uuid_utoa (gfid)); diff --git a/xlators/storage/posix/src/posix.c b/xlators/storage/posix/src/posix.c index 6a623c0a9e8..ab512af15ac 100644 --- a/xlators/storage/posix/src/posix.c +++ b/xlators/storage/posix/src/posix.c @@ -3924,7 +3924,7 @@ init (xlator_t *this) other than that of root '/' */ ret = sys_lgetxattr (dir_data->data, "trusted.gfid", gfid, 16); if (ret == 16) { - if (__is_root_gfid (gfid) != 0) { + if (!__is_root_gfid (gfid)) { gf_log (this->name, GF_LOG_WARNING, "%s: gfid (%s) is not that of glusterfs '/' ", dir_data->data, uuid_utoa (gfid)); |