grpc-proxy/main-grpc: Don't fall trough assuming we have a wl_display 17/29317/3
authorMarius Vlad <marius.vlad@collabora.com>
Fri, 27 Oct 2023 13:16:19 +0000 (16:16 +0300)
committerMarius Vlad <marius.vlad@collabora.com>
Fri, 27 Oct 2023 13:54:38 +0000 (16:54 +0300)
Maybe the third time is a charm to avoid fall through and assume on the
exit path we do have a wl_display.

Bug-AGL: SPEC-4935
Reported-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
Tested-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I4adef5bf72d6b1a32a218bf1cfa6c8ed4450ba48

grpc-proxy/main-grpc.cpp

index a8c1c5e..5569fc8 100644 (file)
@@ -387,7 +387,8 @@ __register_shell_init(void)
 
        wl_display = wl_display_connect(NULL);
        if (!wl_display) {
-               goto err;
+               ret = -1;
+               goto err_failed_display;
        }
        registry = wl_display_get_registry(wl_display);
        sh->wait_for_bound = true;
@@ -416,6 +417,8 @@ __register_shell_init(void)
 err:
        wl_registry_destroy(registry);
        wl_display_disconnect(wl_display);
+
+err_failed_display:
        delete sh;
        return ret;
 }