summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorarao <arao@redhat.com>2015-02-11 18:58:08 +0530
committerVijay Bellur <vbellur@redhat.com>2015-04-10 11:24:52 +0000
commit320455bddfbadcfbe1b173b0de54b3b25bf228f7 (patch)
tree80eea42000e750d165d46b47538abda489f308ce
parentd06e6bac4b5aa6d0fbf1660b92c4100de8f79e68 (diff)
io-stats: Fixing dereference after null check.
CID: 1124492 If 'this' pointer fails to exist then the component is hard-coded in the gf_log if not it is dereferenced. Change-Id: I988137d4f5ac4c9aedef7cef0c75b167a8a5c59f BUG: 789278 Signed-off-by: arao <arao@redhat.com> Reviewed-on: http://review.gluster.org/9639 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
-rw-r--r--xlators/debug/io-stats/src/io-stats.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/debug/io-stats/src/io-stats.c b/xlators/debug/io-stats/src/io-stats.c
index 7acc3803107..f79db1ad19a 100644
--- a/xlators/debug/io-stats/src/io-stats.c
+++ b/xlators/debug/io-stats/src/io-stats.c
@@ -2799,7 +2799,8 @@ reconfigure (xlator_t *this, dict_t *options)
ret = 0;
out:
- gf_log (this->name, GF_LOG_DEBUG, "reconfigure returning %d", ret);
+ gf_log (this ? this->name : "io-stats",
+ GF_LOG_DEBUG, "reconfigure returning %d", ret);
return ret;
}