On 11/08/2012 12:26 AM, Vinson Lee wrote:
Fixes resource leak defect reported by Coverity.

Signed-off-by: Vinson Lee<v...@freedesktop.org>
---
  src/gallium/drivers/svga/svga_resource_buffer.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/src/gallium/drivers/svga/svga_resource_buffer.c 
b/src/gallium/drivers/svga/svga_resource_buffer.c
index 87a4a9b..3aa6644 100644
--- a/src/gallium/drivers/svga/svga_resource_buffer.c
+++ b/src/gallium/drivers/svga/svga_resource_buffer.c
@@ -206,6 +206,8 @@ svga_buffer_transfer_map(struct pipe_context *pipe,
        ++sbuf->map.count;
        map += transfer->box.x;
        *ptransfer = transfer;
+   } else {
+      FREE(transfer);
     }

     return map;

Reviewed-by: Brian Paul <bri...@vmware.com>

I actually fixed this a couple weeks ago in-house but didn't propagate it out yet. Thanks.

-Brina
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to