summaryrefslogtreecommitdiffstats
path: root/libglusterfs
diff options
context:
space:
mode:
authorZhou Zhengping <johnzzpcrystal@gmail.com>2016-07-22 04:09:15 +0800
committerJeff Darcy <jdarcy@redhat.com>2016-07-22 08:12:31 -0700
commitb76ac27f6fe2a183c350f236e3da16f38189e4a8 (patch)
treeabf9dc5c61db79a32b58ee804ccb854fd800707f /libglusterfs
parent99e1686638c26d1a3bb7d66c8a7fd3d93d07e75d (diff)
core: coverity, NULL potinter check
iobuf_get_page_aligned should check the return pointer after calling iobuf_get2. CID: 1357868 Change-Id: I3aa5b09bce2225aa70cf3c60a6b6f4e4eca1dee6 BUG: 1358936 Signed-off-by: Zhou Zhengping <johnzzpcrystal@gmail.com> Reviewed-on: http://review.gluster.org/14976 Smoke: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
Diffstat (limited to 'libglusterfs')
-rw-r--r--libglusterfs/src/iobuf.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/libglusterfs/src/iobuf.c b/libglusterfs/src/iobuf.c
index 17cd68fc206..76584fc9cde 100644
--- a/libglusterfs/src/iobuf.c
+++ b/libglusterfs/src/iobuf.c
@@ -692,6 +692,8 @@ iobuf_get_page_aligned (struct iobuf_pool *iobuf_pool, size_t page_size,
}
iobuf = iobuf_get2 (iobuf_pool, req_size + align_size);
+ if (!iobuf)
+ return NULL;
/* If std allocation was used, then free_ptr will be non-NULL. In this
* case, we do not want to modify the original free_ptr.
* On the other hand, if the buf was gotten through the available