ecore-wl2: Fix resource leak
authorChris Michael <cp.michael@samsung.com>
Sun, 26 Nov 2017 17:12:53 +0000 (12:12 -0500)
committerChris Michael <cp.michael@samsung.com>
Sun, 26 Nov 2017 17:12:53 +0000 (12:12 -0500)
Coverity detected a resource leak here because we were not freeing the
malloc'd 'obo' variable.

Fixes Coverity CID1382907

Signed-off-by: Chris Michael <cp.michael@samsung.com>
src/lib/ecore_wl2/ecore_wl2_buffer.c

index a8e5bec..0403fd0 100644 (file)
@@ -428,7 +428,12 @@ _vc4_alloc(Buffer_Manager *self EINA_UNUSED, const char *name EINA_UNUSED, int w
    memset(&bo, 0, sizeof(bo));
    bo.size = size;
    ret = ioctl(drm_fd, DRM_IOCTL_VC4_CREATE_BO, &bo);
-   if (ret) return NULL;
+   if (ret)
+     {
+        free(obo);
+        return NULL;
+     }
+
    obo->handle = bo.handle;
    obo->size = size;
    /* First try to allocate an mmapable buffer with O_RDWR,
@@ -449,6 +454,7 @@ err:
    memset(&cl, 0, sizeof(cl));
    cl.handle = bo.handle;
    ioctl(drm_fd, DRM_IOCTL_GEM_CLOSE, &cl);
+   free(obo);
    return NULL;
 }