diff options
author | Constantine Kharlamov <[email protected]> | 2017-03-20 21:16:25 +0300 |
---|---|---|
committer | Marek Olšák <[email protected]> | 2017-03-20 23:23:50 +0100 |
commit | 99d400b78f03ce33730b7ac8afa5077d6bb31893 (patch) | |
tree | 2ead1dc3fa231414c32620a0182a8eb7b75b24fd /src/glx/indirect_vertex_program.c | |
parent | 827ae79b2c6cf53e26b3467e4c3965ce6acab3c6 (diff) |
r600g/sb: Fix memory leak by reworking uses list (rebased)
The author is Heiko Przybyl(CC'ing), the patch is rebased on top of Bartosz Tomczyk's one per Dieter Nützel's comment.
Tested-by: Constantine Charlamov <[email protected]>
v2: Resend the patch again through git-email. The prev. rebase was sent
through Thunderbird, which screwed up tab characters, making the patch
not apply.
--------------
When fixing the stalls on evergreen I introduced leaking of the useinfo
structure(s). Sorry. Instead of allocating a new object to hold 3 values
where only one is actually used, rework the list to just store the node
pointer. Thus no allocating and deallocation is needed. Since use_info
and use_kind aren't used anywhere, drop them and reduce code complexity.
This might also save some small amount of cycles.
Thanks to Bartosz Tomczyk for finding the bug.
Reported-by: Bartosz Tomczyk <bartosz.tomczyk86 at gmail.com <https://lists.freedesktop.org/mailman/listinfo/mesa-dev>>
Signed-off-by: Heiko Przybyl <lil_tux at web.de <https://lists.freedesktop.org/mailman/listinfo/mesa-dev>>
Supersedes: https://patchwork.freedesktop.org/patch/135852
Signed-off-by: Marek Olšák <[email protected]>
Tested-by: Dieter Nützel <[email protected]>
Diffstat (limited to 'src/glx/indirect_vertex_program.c')
0 files changed, 0 insertions, 0 deletions