summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorColomban Wendling <[email protected]>2023-11-15 15:31:27 +0100
committerraveit65 <[email protected]>2024-02-04 18:37:58 +0100
commitd437f5bba6a55ef585a2d51beadf68d71f7f7221 (patch)
treecee8a33f187d8dc849cfb48a9c569aa05d4037c8
parent62fabfc793b545250dafaf37f75650cbfdce095f (diff)
downloadmate-panel-d437f5bba6a55ef585a2d51beadf68d71f7f7221.tar.bz2
mate-panel-d437f5bba6a55ef585a2d51beadf68d71f7f7221.tar.xz
Avoid a redundant NULL check
It's probably not necessary to perform the NULL check at all as the only code paths that could make `list` NULL are already pretty dramatically broken, but as we have a NULL check move everything relevant inside it.
-rw-r--r--mate-panel/panel-profile.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/mate-panel/panel-profile.c b/mate-panel/panel-profile.c
index 06d096c2..b637563a 100644
--- a/mate-panel/panel-profile.c
+++ b/mate-panel/panel-profile.c
@@ -1593,12 +1593,13 @@ panel_profile_load_list (GSettings *settings,
list = g_settings_get_strv (settings, key);
- for (gint i = 0; list[i]; i++) {
- load_handler (list[i]);
- }
+ if (list) {
+ for (gint i = 0; list[i]; i++) {
+ load_handler (list[i]);
+ }
- if (list)
g_strfreev (list);
+ }
}
static void