radv/winsys: Deal with realloc failures in BO lists.

Otherwise if realloc fails we silently try to use it. Make recording fail instead.

CC: <mesa-stable@lists.freedesktop.org>
Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5578>
This commit is contained in:
Bas Nieuwenhuizen 2020-06-20 20:34:59 +02:00 committed by Marge Bot
parent 519ddfd312
commit 04765e6a9a

View file

@ -558,13 +558,20 @@ static void radv_amdgpu_cs_add_buffer_internal(struct radv_amdgpu_cs *cs,
unsigned hash;
int index = radv_amdgpu_cs_find_buffer(cs, bo);
if (index != -1)
if (index != -1 || cs->failed)
return;
if (cs->num_buffers == cs->max_num_buffers) {
unsigned new_count = MAX2(1, cs->max_num_buffers * 2);
cs->handles = realloc(cs->handles, new_count * sizeof(struct drm_amdgpu_bo_list_entry));
cs->max_num_buffers = new_count;
struct drm_amdgpu_bo_list_entry *new_entries =
realloc(cs->handles, new_count * sizeof(struct drm_amdgpu_bo_list_entry));
if (new_entries) {
cs->max_num_buffers = new_count;
cs->handles = new_entries;
} else {
cs->failed = true;
return;
}
}
cs->handles[cs->num_buffers].bo_handle = bo;