diff options
author | Amar Tumballi <amarts@redhat.com> | 2019-05-20 11:11:39 +0530 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2019-06-04 10:30:29 +0000 |
commit | e7aeab3063ac5645136303278b477d7de35266c0 (patch) | |
tree | 73ab2e1ea4e26b1019318178d114c3d89dc34301 /xlators/features/quota/src/quota.c | |
parent | abc4d4cfa01d922c46ebb1abfa5d801ba8c6b5c8 (diff) |
across: clang-scan: fix NULL dereferencing warnings
All these checks are done after analyzing clang-scan report produced
by the CI job @ https://build.gluster.org/job/clang-scan
updates: bz#1622665
Change-Id: I590305af4ceb779be952974b2a36066ffc4865ca
Signed-off-by: Amar Tumballi <amarts@redhat.com>
Diffstat (limited to 'xlators/features/quota/src/quota.c')
-rw-r--r-- | xlators/features/quota/src/quota.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/xlators/features/quota/src/quota.c b/xlators/features/quota/src/quota.c index a0c236d4cf6..1335386a197 100644 --- a/xlators/features/quota/src/quota.c +++ b/xlators/features/quota/src/quota.c @@ -1921,10 +1921,12 @@ quota_writev(call_frame_t *frame, xlator_t *this, fd_t *fd, par_inode = do_quota_check_limit(frame, fd->inode, this, dentry, _gf_false); if (par_inode == NULL) { - /* remove stale entry from inode ctx */ - quota_dentry_del(ctx, dentry->name, dentry->par); - parents--; - fail_count++; + if (ctx) { + /* remove stale entry from inode ctx */ + quota_dentry_del(ctx, dentry->name, dentry->par); + parents--; + fail_count++; + } } else { inode_unref(par_inode); } |