diff options
| author | Anuradha <atalur@redhat.com> | 2015-05-25 11:07:27 +0530 |
|---|---|---|
| committer | Raghavendra G <rgowdapp@redhat.com> | 2015-06-01 22:25:37 -0700 |
| commit | f3a340694fcb195aa8b546578c348b41fb2208d1 (patch) | |
| tree | 9e5445d210c3c819d0d9da1cb458169e6d7491ce | |
| parent | 751c4583bbaa59ebfe492ab6ecfab3108711f4c5 (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.c | 2 |
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; } |