]> git.meshlink.io Git - meshlink/commitdiff
Non-zero exit code when reloading config file fails after SIGHUP.
authorGuus Sliepen <guus@tinc-vpn.org>
Fri, 2 Aug 2013 21:46:19 +0000 (23:46 +0200)
committerGuus Sliepen <guus@tinc-vpn.org>
Fri, 2 Aug 2013 21:50:44 +0000 (23:50 +0200)
When reloading the configuration file via the tinc command, the user will get
an error message if reloading has failed. However, no such warning exists when
sending a HUP signal. Previously, tincd would exit in both cases, but with a
zero exit code. Now it will exit with code 1 when reloading fails after a
SIGHUP, but tincd will keep running if it is signaled via the tinc command.
Instead, the tinc command will exit with a non-zero exit code.

src/conf.c
src/net.c

index 71e45092e0fa45e209302e1e8b5ed813b6050524..3c645197275c74d351945783ff3fbb9090cd95ca 100644 (file)
@@ -373,9 +373,10 @@ bool read_server_config(void) {
        read_config_options(config_tree, NULL);
 
        xasprintf(&fname, "%s" SLASH "tinc.conf", confbase);
+       errno = 0;
        x = read_config_file(config_tree, fname);
 
-       if(!x)
+       if(!x && errno)
                logger(DEBUG_ALWAYS, LOG_ERR, "Failed to read `%s': %s", fname, strerror(errno));
 
        free(fname);
index 0b43d5acfceec9480e8b0b4bab8b4bc518bc2a42..bf6cfcf177d58a18572a4947e62a45e11f73c617 100644 (file)
--- a/src/net.c
+++ b/src/net.c
@@ -298,7 +298,8 @@ static void sigterm_handler(void *data) {
 static void sighup_handler(void *data) {
        logger(DEBUG_ALWAYS, LOG_NOTICE, "Got %s signal", strsignal(((signal_t *)data)->signum));
        reopenlogger();
-       reload_configuration();
+       if(reload_configuration())
+               exit(1);
 }
 
 static void sigalrm_handler(void *data) {
@@ -316,8 +317,7 @@ int reload_configuration(void) {
        init_configuration(&config_tree);
 
        if(!read_server_config()) {
-               logger(DEBUG_ALWAYS, LOG_ERR, "Unable to reread configuration file, exitting.");
-               event_exit();
+               logger(DEBUG_ALWAYS, LOG_ERR, "Unable to reread configuration file.");
                return EINVAL;
        }