[Asterisk-code-review] core: Fix handling of restart from remote console. (asterisk[16])

Corey Farrell asteriskteam at digium.com
Thu Nov 15 05:33:54 CST 2018


Corey Farrell has uploaded this change for review. ( https://gerrit.asterisk.org/10647


Change subject: core: Fix handling of restart from remote console.
......................................................................

core: Fix handling of restart from remote console.

We cannot use need_el_end and SIGURG when restarting.  Instead we need
to run el_end within the SIGHUP restartnow handler.

ASTERISK-28158

Change-Id: Ia852276363c81bdcf1aa29eb4558c5c2fa1218a0
---
M main/asterisk.c
1 file changed, 9 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/47/10647/1

diff --git a/main/asterisk.c b/main/asterisk.c
index 12e6255..cf50b3f 100644
--- a/main/asterisk.c
+++ b/main/asterisk.c
@@ -1673,9 +1673,15 @@
 static void _hup_handler(int num)
 {
 	int save_errno = errno;
-	printf("Received HUP signal -- Reloading configs\n");
-	if (restartnow)
+
+	if (restartnow) {
+		if (el) {
+			el_end(el);
+		}
 		execvp(_argv[0], _argv);
+	}
+
+	printf("Received HUP signal -- Reloading configs\n");
 	sig_flags.need_reload = 1;
 	if (ast_alertpipe_write(sig_alert_pipe)) {
 		fprintf(stderr, "hup_handler: write() failed: %s\n", strerror(errno));
@@ -2016,7 +2022,7 @@
 			if (el_hist != NULL) {
 				history_end(el_hist);
 			}
-		} else {
+		} else if (!restart) {
 			sig_flags.need_el_end = 1;
 			pthread_kill(consolethread, SIGURG);
 		}

-- 
To view, visit https://gerrit.asterisk.org/10647
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia852276363c81bdcf1aa29eb4558c5c2fa1218a0
Gerrit-Change-Number: 10647
Gerrit-PatchSet: 1
Gerrit-Owner: Corey Farrell <git at cfware.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181115/84ec0eda/attachment.html>


More information about the asterisk-code-review mailing list