diff options
author | Harpreet Lalwani <hlalwani@redhat.com> | 2018-10-04 16:09:08 +0530 |
---|---|---|
committer | N Balachandran <nbalacha@redhat.com> | 2018-10-31 09:42:14 +0000 |
commit | d2d3c37fa8efca74e375b8a0dc7fa720c3b84dcc (patch) | |
tree | a5adbe9849a1e40e41bbee046369f080d10ce8e5 /xlators/cluster/dht/src/dht-helper.c | |
parent | 9be6bf3d90e3783b3ba559c93d41b933f8d53f03 (diff) |
cluster/dht: NULL pointer dereferencing clang fix
Dereferencing NUll pointers this,local and stbuf.
1.Replaced this->name with "dht".
2.Removed GF_VALIDATE_OR_GOTO.
3.Removed the check for "stbuf" and "this".
Updates: bz#1622665
Change-Id: Id2fb2270d5ec37b76fa2aae1f1c8dca72dcc728a
Signed-off-by: Harpreet Lalwani <hlalwani@redhat.com>
Diffstat (limited to 'xlators/cluster/dht/src/dht-helper.c')
-rw-r--r-- | xlators/cluster/dht/src/dht-helper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/cluster/dht/src/dht-helper.c b/xlators/cluster/dht/src/dht-helper.c index 4764ac5fdd3..22ce8bc4d4c 100644 --- a/xlators/cluster/dht/src/dht-helper.c +++ b/xlators/cluster/dht/src/dht-helper.c @@ -938,7 +938,7 @@ dht_subvol_get_cached(xlator_t *this, inode_t *inode) dht_layout_t *layout = NULL; xlator_t *subvol = NULL; - GF_VALIDATE_OR_GOTO(this->name, this, out); + GF_VALIDATE_OR_GOTO("dht", this, out); GF_VALIDATE_OR_GOTO(this->name, inode, out); layout = dht_layout_get(this, inode); |