winevulkan: Free all command buffers when vkAllocateCommandBuffers() fails.

Signed-off-by: Józef Kucia <jkucia@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Józef Kucia 2018-08-30 12:22:34 +02:00 committed by Alexandre Julliard
parent 29d49fb8a3
commit b62d4ea6ed

View File

@ -540,25 +540,24 @@ VkResult WINAPI wine_vkAllocateCommandBuffers(VkDevice device,
buffers[i]->base.loader_magic = VULKAN_ICD_MAGIC_VALUE; buffers[i]->base.loader_magic = VULKAN_ICD_MAGIC_VALUE;
buffers[i]->device = device; buffers[i]->device = device;
list_add_tail(&pool->command_buffers, &buffers[i]->pool_link);
res = device->funcs.p_vkAllocateCommandBuffers(device->device, res = device->funcs.p_vkAllocateCommandBuffers(device->device,
&allocate_info_host, &buffers[i]->command_buffer); &allocate_info_host, &buffers[i]->command_buffer);
if (res != VK_SUCCESS) if (res != VK_SUCCESS)
{ {
ERR("Failed to allocate command buffer, res=%d.\n", res); ERR("Failed to allocate command buffer, res=%d.\n", res);
buffers[i]->command_buffer = VK_NULL_HANDLE;
break; break;
} }
list_add_tail(&pool->command_buffers, &buffers[i]->pool_link);
} }
if (res != VK_SUCCESS) if (res != VK_SUCCESS)
{ {
wine_vk_free_command_buffers(device, pool, i, buffers); wine_vk_free_command_buffers(device, pool, i + 1, buffers);
memset(buffers, 0, allocate_info->commandBufferCount * sizeof(*buffers)); memset(buffers, 0, allocate_info->commandBufferCount * sizeof(*buffers));
return res;
} }
return VK_SUCCESS; return res;
} }
void WINAPI wine_vkCmdExecuteCommands(VkCommandBuffer buffer, uint32_t count, void WINAPI wine_vkCmdExecuteCommands(VkCommandBuffer buffer, uint32_t count,