diff options
author | Eric Anholt <[email protected]> | 2015-06-17 23:49:19 -0700 |
---|---|---|
committer | Eric Anholt <[email protected]> | 2015-06-17 23:53:49 -0700 |
commit | dc1fbad2eb5454ed36a066d2a69b575cd5a8abaf (patch) | |
tree | 56b941964c5e908d29e72a22f9fae56c10c81553 /src/gallium/drivers/vc4 | |
parent | 62d153ea37b1bf572c39aab8ec46099fc903362d (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.
Diffstat (limited to 'src/gallium/drivers/vc4')
-rw-r--r-- | src/gallium/drivers/vc4/vc4_bufmgr.c | 5 |
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 |