summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAnuradha <atalur@redhat.com>2015-05-25 11:07:27 +0530
committerRaghavendra G <rgowdapp@redhat.com>2015-06-01 22:25:37 -0700
commitf3a340694fcb195aa8b546578c348b41fb2208d1 (patch)
tree9e5445d210c3c819d0d9da1cb458169e6d7491ce
parent751c4583bbaa59ebfe492ab6ecfab3108711f4c5 (diff)
features/quota : Do unwind if postbuf is NULL
If postbuf in quota_writev_cbk is NULL directly an unwind should be done. Trying to dereference it will lead to a crash. Change-Id: Idba6ce3cd1bbf37ede96c7f17d01007d6c07057a BUG: 1221577 Signed-off-by: Anuradha <atalur@redhat.com> Reviewed-on: http://review.gluster.org/10898 Tested-by: NetBSD Build System <jenkins@build.gluster.org> Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Raghavendra G <rgowdapp@redhat.com> Tested-by: Raghavendra G <rgowdapp@redhat.com>
-rw-r--r--xlators/features/quota/src/quota.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/features/quota/src/quota.c b/xlators/features/quota/src/quota.c
index ba5e27bd235..6d05273026a 100644
--- a/xlators/features/quota/src/quota.c
+++ b/xlators/features/quota/src/quota.c
@@ -1502,7 +1502,7 @@ quota_writev_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
local = frame->local;
- if ((op_ret < 0) || (local == NULL)) {
+ if ((op_ret < 0) || (local == NULL) || (postbuf == NULL)) {
goto out;
}