From 956b689d6ab18ad78f53f15cdeb7606af98e9e5e Mon Sep 17 00:00:00 2001 From: Greg Depoire--Ferrer Date: Sun, 5 Jun 2022 10:37:17 +0200 Subject: [PATCH] swaynag: move close_button up to fix SIGSEGV When swaynag_parse_options encounters '--dismiss-button' (or its shorthand '-s'), it sets the text of the first button in the swaynag.buttons list, which is expected to exist and to be the dismiss button, to the one passed by the user. Commit 4780afb68b4ee2cdf0e4925f40cf885819f8a74a ("swaynag: statically allocate button_close, and move declaration") moved the list initialization to after swaynag_parse_options is called which made that code fail. For example, the command 'swaynag --dismiss-button Dismiss' crashes and 'swaynag --message Message --button Yes "" --dismiss-button Dismiss' shows the wrong buttons. Move it back to before swaynag_parse_options is called. --- swaynag/main.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/swaynag/main.c b/swaynag/main.c index 56e4950b..2ce37831 100644 --- a/swaynag/main.c +++ b/swaynag/main.c @@ -29,6 +29,11 @@ int main(int argc, char **argv) { wl_list_init(&swaynag.outputs); wl_list_init(&swaynag.seats); + struct swaynag_button button_close = { 0 }; + button_close.text = strdup("X"); + button_close.type = SWAYNAG_ACTION_DISMISS; + list_add(swaynag.buttons, &button_close); + char *config_path = NULL; bool debug = false; status = swaynag_parse_options(argc, argv, NULL, NULL, NULL, @@ -85,11 +90,6 @@ int main(int argc, char **argv) { swaynag_types_free(types); - struct swaynag_button button_close = { 0 }; - button_close.text = strdup("X"); - button_close.type = SWAYNAG_ACTION_DISMISS; - list_add(swaynag.buttons, &button_close); - if (swaynag.details.message) { list_add(swaynag.buttons, &swaynag.details.button_details); }