summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Anholt <[email protected]>2015-06-17 23:49:19 -0700
committerEric Anholt <[email protected]>2015-06-17 23:53:49 -0700
commitdc1fbad2eb5454ed36a066d2a69b575cd5a8abaf (patch)
tree56b941964c5e908d29e72a22f9fae56c10c81553
parent62d153ea37b1bf572c39aab8ec46099fc903362d (diff)
vc4: Fix memory leak from simple_list conversion.
I accidentally shadowed the outside declaration, so we always returned NULL even when we'd found something in the cache.
-rw-r--r--src/gallium/drivers/vc4/vc4_bufmgr.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/src/gallium/drivers/vc4/vc4_bufmgr.c b/src/gallium/drivers/vc4/vc4_bufmgr.c
index eef7e9dc16f..cbdb9e89cf6 100644
--- a/src/gallium/drivers/vc4/vc4_bufmgr.c
+++ b/src/gallium/drivers/vc4/vc4_bufmgr.c
@@ -87,9 +87,8 @@ vc4_bo_from_cache(struct vc4_screen *screen, uint32_t size, const char *name)
struct vc4_bo *bo = NULL;
pipe_mutex_lock(cache->lock);
if (!list_empty(&cache->size_list[page_index])) {
- struct vc4_bo *bo = LIST_ENTRY(struct vc4_bo,
- cache->size_list[page_index].next,
- size_list);
+ bo = LIST_ENTRY(struct vc4_bo, cache->size_list[page_index].next,
+ size_list);
/* Check that the BO has gone idle. If not, then we want to
* allocate something new instead, since we assume that the