diff options
| author | Austin Hu <austin.hu@intel.com> | 2017-05-09 23:10:56 +0000 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2017-05-09 23:10:56 +0000 |
| commit | a5760c68f2880c7451d4fc84b629c9142fe65dae (patch) | |
| tree | 48c380536179899be66fe245127e0bc1a97da7d5 | |
| parent | ac747113d4f6739b1462ca7fb40f2091691e209b (diff) | |
| parent | 8cae226f534a504f385084d1b04f4af466a16990 (diff) | |
| download | libwsbm-a5760c68f2880c7451d4fc84b629c9142fe65dae.tar.gz | |
Fixed Klocwork erros. am: ba6cd22cc9oreo-mr1-devoreo-dr1-dev
am: 8cae226f53
Change-Id: Ifac4875c488a9d910172931375d69b581189fa38
| -rw-r--r-- | src/wsbm_manager.c | 13 | ||||
| -rw-r--r-- | src/wsbm_slabpool.c | 4 |
2 files changed, 12 insertions, 5 deletions
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 @@ wsbmBOCreateList(int target, int hasKernelBuffers) 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; 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 @@ wsbmAllocSlab(struct _WsbmSlabSizeHeader *header) } numBuffers = slab->kbo->actualSize / header->bufSize; + if (!numBuffers) { + ret = -ENOMEM; + goto out_err1; + } slab->buffers = calloc(numBuffers, sizeof(*slab->buffers)); if (!slab->buffers) { |