diff options
author | Michael Scherer <misc@redhat.com> | 2017-02-23 21:35:45 +0100 |
---|---|---|
committer | Shyamsundar Ranganathan <srangana@redhat.com> | 2017-02-26 11:31:20 -0500 |
commit | 235ae3528d4e8a914fc208ff05580336c865727c (patch) | |
tree | 44fd1ed2a2579be1f549b8cc0585a4ca06410962 | |
parent | 730f520a1ee246a5f4a08a321b6f97bdf93536dc (diff) |
dnt: Fix crash when showing the error message
Since this is NULL, this->name would crash the
daemon if a incorrect rebal_entry is passed.
Found by coverity scan.
Change-Id: Ieddf1ef097d13711ab8ec9cd24e125914d2e7245
BUG: 789278
Signed-off-by: Michael Scherer <misc@redhat.com>
Reviewed-on: https://review.gluster.org/16740
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Tested-by: Michael Scherer <misc@fedoraproject.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
-rw-r--r-- | xlators/cluster/dht/src/dht-rebalance.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c index 41a50cff3cc..c851266588d 100644 --- a/xlators/cluster/dht/src/dht-rebalance.c +++ b/xlators/cluster/dht/src/dht-rebalance.c @@ -2115,7 +2115,7 @@ gf_defrag_migrate_single_file (void *opaque) rebal_entry = (struct dht_container *)opaque; if (!rebal_entry) { - gf_log (this->name, GF_LOG_ERROR, "rebal_entry is NULL"); + gf_log ("DHT", GF_LOG_ERROR, "rebal_entry is NULL"); ret = -1; goto out; } |