[PATCH 09/11] winevulkan: Avoid goto in wine_vkCreateDevice().
Józef Kucia
jkucia at codeweavers.com
Thu Mar 29 07:53:56 CDT 2018
Signed-off-by: Józef Kucia <jkucia at codeweavers.com>
---
dlls/winevulkan/vulkan.c | 17 +++++++----------
1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/dlls/winevulkan/vulkan.c b/dlls/winevulkan/vulkan.c
index 4c9874c78473..7a7a334e9c38 100644
--- a/dlls/winevulkan/vulkan.c
+++ b/dlls/winevulkan/vulkan.c
@@ -531,8 +531,9 @@ VkResult WINAPI wine_vkCreateDevice(VkPhysicalDevice phys_dev,
&create_info_host, NULL /* allocator */, &object->device);
if (res != VK_SUCCESS)
{
- ERR("Failed to create device\n");
- goto err;
+ ERR("Failed to create device.\n");
+ wine_vk_device_free(object);
+ return res;
}
object->phys_dev = phys_dev;
@@ -559,8 +560,8 @@ VkResult WINAPI wine_vkCreateDevice(VkPhysicalDevice phys_dev,
object->queues = heap_calloc(max_queue_families, sizeof(*object->queues));
if (!object->queues)
{
- res = VK_ERROR_OUT_OF_HOST_MEMORY;
- goto err;
+ wine_vk_device_free(object);
+ return VK_ERROR_OUT_OF_HOST_MEMORY;
}
for (i = 0; i < create_info_host.queueCreateInfoCount; i++)
@@ -573,18 +574,14 @@ VkResult WINAPI wine_vkCreateDevice(VkPhysicalDevice phys_dev,
object->queues[family_index] = wine_vk_device_alloc_queues(object, family_index, queue_count);
if (!object->queues[family_index])
{
- res = VK_ERROR_OUT_OF_HOST_MEMORY;
ERR("Failed to allocate memory for queues\n");
- goto err;
+ wine_vk_device_free(object);
+ return VK_ERROR_OUT_OF_HOST_MEMORY;
}
}
*device = object;
return VK_SUCCESS;
-
-err:
- wine_vk_device_free(object);
- return res;
}
VkResult WINAPI wine_vkCreateInstance(const VkInstanceCreateInfo *create_info,
--
2.16.1
More information about the wine-devel
mailing list