[gdm/fix-fallback-mode: 26/35] local-display-factory: Add missing continue statements




commit 6f2a658d8f5652556d7379c1330e00d1bf6d3934
Author: Ray Strode <rstrode redhat com>
Date:   Tue Jul 27 07:03:47 2021 -0400

    local-display-factory: Add missing continue statements
    
    commit f4922c046607c45d76e2911aa8f133d0ad4f9223 tried to
    fix an overrun in the code, but it neglected to add
    
    "continue" statements to the loops, so it was stuffing
    two different values into the same element of an array,
    which leads to the wrong session type getting preference.
    
    This commit fixes that.

 daemon/gdm-local-display-factory.c | 2 ++
 1 file changed, 2 insertions(+)
---
diff --git a/daemon/gdm-local-display-factory.c b/daemon/gdm-local-display-factory.c
index f2da3b6e4..141d64c6b 100644
--- a/daemon/gdm-local-display-factory.c
+++ b/daemon/gdm-local-display-factory.c
@@ -246,6 +246,7 @@ gdm_local_display_factory_get_session_type (GdmLocalDisplayFactory *factory,
                     if (gdm_settings_direct_get_boolean (GDM_KEY_WAYLAND_ENABLE, &wayland_enabled)) {
                             if (wayland_enabled && g_file_test ("/usr/bin/Xwayland", 
G_FILE_TEST_IS_EXECUTABLE)) {
                                     session_types[i] = "wayland";
+                                    continue;
                             }
                     }
             }
@@ -257,6 +258,7 @@ gdm_local_display_factory_get_session_type (GdmLocalDisplayFactory *factory,
                     if (gdm_settings_direct_get_boolean (GDM_KEY_XORG_ENABLE, &xorg_enabled)) {
                             if (xorg_enabled && g_file_test ("/usr/bin/Xorg", G_FILE_TEST_IS_EXECUTABLE)) {
                                     session_types[i] = "x11";
+                                    continue;
                             }
                     }
             }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]