]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
asterisk.c: Fix sending incorrect messages to systemd notify
authorIvan Poddubny <ivan.poddubny@gmail.com>
Sun, 5 May 2024 12:53:11 +0000 (14:53 +0200)
committerIvan Poddubny <ivan.poddubny@gmail.com>
Mon, 6 May 2024 16:08:53 +0000 (16:08 +0000)
Send "RELOADING=1" instead of "RELOAD=1" to follow the format
expected by systemd (see sd_notify(3) man page).

Do not send STOPPING=1 in remote console mode:
attempting to execute "asterisk -rx" by the main process leads to
a warning if NotifyAccess=main (the default) or to a forced termination
if NotifyAccess=all.

main/asterisk.c
main/loader.c

index 0d306fdeebe28f6a425414d791135f6f79205852..0f7b9f7e7cf67c2151be8b5361d33b8056529a88 100644 (file)
@@ -2050,7 +2050,7 @@ static void really_quit(int num, shutdown_nice_t niceness, int restart)
                run_cleanups = 0;
        }
 
-       if (!restart) {
+       if (!restart && !ast_opt_remote) {
                ast_sd_notify("STOPPING=1");
        }
        if (ast_opt_console || (ast_opt_remote && !ast_opt_exec)) {
index ecad2346e11de67d8ee9b58f53d0592d7333bf65..3ba9737de49a86d2ffaaed12cf0a2ad74cfdfa07 100644 (file)
@@ -1583,7 +1583,7 @@ enum ast_module_reload_result ast_module_reload(const char *name)
                res = AST_MODULE_RELOAD_IN_PROGRESS;
                goto module_reload_exit;
        }
-       ast_sd_notify("RELOAD=1");
+       ast_sd_notify("RELOADING=1");
        ast_lastreloadtime = ast_tvnow();
 
        if (ast_opt_lock_confdir) {