diff options
| author | arao <arao@redhat.com> | 2015-02-10 02:51:43 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2015-03-31 01:32:14 -0700 |
| commit | 6522f059eeec1b9962525b61c4c6640ae1afe85c (patch) | |
| tree | a9d30d3b372be084f0db6f2a4d81b7113949e799 /xlators/features | |
| parent | e59e3a341308a9f47231d0205359da6886bb82be (diff) | |
features/changelog: Fixing Deference after null check
CID: 1124481
The goto label 'out' was trying to
dereference a pointer which might be
null, hence a check on not null is
made before the use of it.
Change-Id: I813e523207b8218a7a28c2b3ac36a8e472e36c89
BUG: 789278
Signed-off-by: arao <arao@redhat.com>
Reviewed-on: http://review.gluster.org/9617
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/features')
| -rw-r--r-- | xlators/features/changelog/src/changelog.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/features/changelog/src/changelog.c b/xlators/features/changelog/src/changelog.c index 6c7a0a80f17..1c7166e52b5 100644 --- a/xlators/features/changelog/src/changelog.c +++ b/xlators/features/changelog/src/changelog.c @@ -2376,7 +2376,7 @@ reconfigure (xlator_t *this, dict_t *options) } else { gf_log (this->name, GF_LOG_DEBUG, "changelog reconfigured"); - if (active_now) + if (active_now && priv) priv->active = _gf_true; } |