diff options
| author | arao <arao@redhat.com> | 2015-02-11 18:58:08 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2015-04-10 11:24:52 +0000 |
| commit | 320455bddfbadcfbe1b173b0de54b3b25bf228f7 (patch) | |
| tree | 80eea42000e750d165d46b47538abda489f308ce | |
| parent | d06e6bac4b5aa6d0fbf1660b92c4100de8f79e68 (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.c | 3 |
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; } |