diff options
| author | Vijay Bellur <vbellur@redhat.com> | 2018-11-07 06:31:19 -0800 | 
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-11-12 03:38:08 +0000 | 
| commit | f68f9106087a3dc4e6da88a09285ac723096d7e9 (patch) | |
| tree | d5889118a47fe27463eb5d7afec8506cd1ac923b /xlators/features/changelog/lib | |
| parent | bcf1e8b07491b48c5372924dbbbad5b8391c6d81 (diff) | |
features/changelog: Avoid a potential null dereference
Addresses CID 1396292
Change-Id: I9141502a12fb3e18c04d07c79da87a5ebb2cfc23
updates: bz#789278
Signed-off-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/features/changelog/lib')
| -rw-r--r-- | xlators/features/changelog/lib/src/gf-changelog.c | 10 | 
1 files changed, 6 insertions, 4 deletions
diff --git a/xlators/features/changelog/lib/src/gf-changelog.c b/xlators/features/changelog/lib/src/gf-changelog.c index 8cfab4fe391..42e20915922 100644 --- a/xlators/features/changelog/lib/src/gf-changelog.c +++ b/xlators/features/changelog/lib/src/gf-changelog.c @@ -160,9 +160,13 @@ gf_changelog_ctx_defaults_init(glusterfs_ctx_t *ctx)      return 0;  free_pool: -    GF_FREE(pool->frame_mem_pool); +    if (pool) { +        GF_FREE(pool->frame_mem_pool); -    GF_FREE(pool->stack_mem_pool); +        GF_FREE(pool->stack_mem_pool); + +        GF_FREE(pool); +    }      GF_FREE(ctx->stub_mem_pool); @@ -174,8 +178,6 @@ free_pool:      GF_FREE(ctx->logbuf_pool); -    GF_FREE(pool); -      GF_FREE(ctx->iobuf_pool);      GF_FREE(ctx->event_pool);  | 
