The dma_bo struct was never freed in the error path. Reported by cppcheck.

Signed-off-by: Thomas DeRensis <[email protected]>
---
 src/radeon_vbo.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/radeon_vbo.c b/src/radeon_vbo.c
index 1924772..6b0d278 100644
--- a/src/radeon_vbo.c
+++ b/src/radeon_vbo.c
@@ -185,6 +185,7 @@ again_alloc:
 
        if (!dma_bo->bo) {
            ErrorF("failure to allocate DMA BO\n");
+           free(dma_bo);
            return NULL;
        }
        insert_at_head(&accel_state->bo_reserved, dma_bo);
-- 
2.1.0

_______________________________________________
xorg-driver-ati mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-driver-ati

Reply via email to