diff options
author | Jason Ekstrand <[email protected]> | 2016-09-07 21:33:48 -0700 |
---|---|---|
committer | Jason Ekstrand <[email protected]> | 2016-09-13 10:44:03 -0700 |
commit | 6ac469a6c33d7b5b6a1e0abcb4761e9ca05fa449 (patch) | |
tree | 7275046ec7c75123227a6bce1b28e1338294334d /src/mapi/mapi.c | |
parent | 3943888c94beca69e575b8d3d1ec7a6cbf474ee4 (diff) |
anv/allocator: Use VG_NOACCESS_WRITE in anv_bo_pool_free
Previously, we were relying on the fact that VALGRIND_MEMPOOL_FREE came
later on in the function to prevent "link->bo = bo" from causing an invalid
write. However, in the case where the size requested by the user is very
small (less than sizeof(struct anv_bo)), this isn't sufficient. Instead,
we should call VALGRIND_MEMPOOL_FREE early and then use VG_NOACCESS_WRITE.
We do, however, have to call VALGRIND_MEMPOOL_FREE after reading bo_in
because it may be stored in the bo itself.
Signed-off-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src/mapi/mapi.c')
0 files changed, 0 insertions, 0 deletions