commit 8cae226f534a504f385084d1b04f4af466a16990
author Austin Hu <austin.hu@intel.com> Tue May 09 21:30:13 2017 +0000
committer android-build-merger <android-build-merger@google.com> Tue May 09 21:30:13 2017 +0000
tree 48c380536179899be66fe245127e0bc1a97da7d5
parent ac747113d4f6739b1462ca7fb40f2091691e209b
parent ba6cd22cc9dbadb14dad331ef6ddc8b1e403a654

Fixed Klocwork erros.
am: ba6cd22cc9

Change-Id: Id0b6c7d39acda75cd07d38c58dd64e9ce64b7a9a

src/wsbm_manager.c

diff --git a/src/wsbm_manager.c b/src/wsbm_manager.c
index 0cbf9de..877abc6 100644
--- a/src/wsbm_manager.c
+++ b/src/wsbm_manager.c
@@ -1015,15 +1015,18 @@
         return NULL;
     list->hasKernelBuffers = hasKernelBuffers;
     if (hasKernelBuffers) {
-	ret = validateCreateList(target, &list->kernelBuffers, 0);
-	if (ret)
-	    return NULL;
+        ret = validateCreateList(target, &list->kernelBuffers, 0);
+        if (ret) {
+            free(list);
+            return NULL;
+        }
     }
 
     ret = validateCreateList(target, &list->userBuffers, 1);
     if (ret) {
-	validateFreeList(&list->kernelBuffers);
-	return NULL;
+        validateFreeList(&list->kernelBuffers);
+        free(list);
+        return NULL;
     }
 
     return list;

src/wsbm_slabpool.c

diff --git a/src/wsbm_slabpool.c b/src/wsbm_slabpool.c
index ec6638b..1e551e3 100644
--- a/src/wsbm_slabpool.c
+++ b/src/wsbm_slabpool.c
@@ -463,6 +463,10 @@
     }
 
     numBuffers = slab->kbo->actualSize / header->bufSize;
+    if (!numBuffers) {
+        ret = -ENOMEM;
+        goto out_err1;
+    }
 
     slab->buffers = calloc(numBuffers, sizeof(*slab->buffers));
     if (!slab->buffers) {