fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-01-06 04:30:10 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

anv: Zero released anv_bo structs

Browse source 
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
This commit is contained in:
Jason Ekstrand 2019-10-30 12:31:12 -05:00
parent b3c0b1b218
commit 0ca0ad1252
1 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/intel/vulkan/anv_allocator.c
View file

@ -1846,7 +1846,18 @@ anv_device_release_bo(struct anv_device *device,
if (!bo->has_fixed_address)
anv_vma_free(device, bo);
anv_gem_close(device, bo->gem_handle);
uint32_t gem_handle = bo->gem_handle;
/* Memset the BO just in case. The refcount being zero should be enough to
* prevent someone from assuming the data is valid but it's safer to just
* stomp to zero just in case. We explicitly do this *before* we close the
* GEM handle to ensure that if anyone allocates something and gets the
* same GEM handle, the memset has already happen and won't stomp all over
* any data they may write in this BO.
*/
memset(bo, 0, sizeof(*bo));
anv_gem_close(device, gem_handle);
/* Don't unlock until we've actually closed the BO. The whole point of
* the BO cache is to ensure that we correctly handle races with creating