[Mesa-dev] [PATCH v2] vulkan: Fix use of uninitialized variable.

robert.foss at collabora.com robert.foss at collabora.com
Wed May 4 12:56:41 UTC 2016


From: Robert Foss <robert.foss at collabora.com>

The return variable was not set for failure paths.
It has now been changed to VK_ERROR_INITIALIZATION_FAILED
for failure paths.

Coverity: 1358944
Signed-off-by: Robert Foss <robert.foss at collabora.com>
---

Changes since v1:
- Started using vk_error().

 src/intel/vulkan/anv_wsi_wayland.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/intel/vulkan/anv_wsi_wayland.c b/src/intel/vulkan/anv_wsi_wayland.c
index 6f25eaf..43606f6 100644
--- a/src/intel/vulkan/anv_wsi_wayland.c
+++ b/src/intel/vulkan/anv_wsi_wayland.c
@@ -776,12 +776,16 @@ wsi_wl_surface_create_swapchain(VkIcdSurfaceBase *icd_surface,
    chain->queue = NULL;
 
    chain->display = wsi_wl_get_display(device->instance, surface->display);
-   if (!chain->display)
+   if (!chain->display) {
+      result = vk_error(VK_ERROR_INITIALIZATION_FAILED);
       goto fail;
+   }
 
    chain->queue = wl_display_create_queue(chain->display->display);
-   if (!chain->queue)
+   if (!chain->queue) {
+      result = vk_error(VK_ERROR_INITIALIZATION_FAILED);
       goto fail;
+   }
 
    for (uint32_t i = 0; i < chain->image_count; i++) {
       result = wsi_wl_image_init(chain, &chain->images[i], pAllocator);
-- 
2.5.0



More information about the mesa-dev mailing list