Fix #258: Set fallback alternative color before trying to allocate it.

1 files changed, 5 insertions, 4 deletions

diff --git a/source/helper.c b/source/helper.c
index 8a77d398..7157799f 100644
--- a/source/helper.c
+++ b/source/helper.c
@@ -539,6 +539,11 @@ void remove_pid_file ( int fd )
  */
 void config_sanity_check ( Display *display )
 {
+    // If alternative row is not set, copy the normal background color.
+    // Do this at the beginning as we might use it in the error dialog.
+    if ( config.menu_bg_alt == NULL ) {
+        config.menu_bg_alt = config.menu_bg;
+    }
     int     found_error = FALSE;
     GString *msg        = g_string_new (
         "<big><b>The configuration failed to validate:</b></big>\n" );
@@ -612,10 +617,6 @@ void config_sanity_check ( Display *display )
     }
 
     g_string_free ( msg, TRUE );
-    // If alternative row is not set, copy the normal background color.
-    if ( config.menu_bg_alt == NULL ) {
-        config.menu_bg_alt = config.menu_bg;
-    }
 }
 
 int is_not_ascii ( const char * str )