Ack, code review. Thanks/Minh

On 9/2/19 2:11 pm, Gary Lee wrote:
fix warnings about unused variables and add SA_RESTART
---
  src/base/daemon.c | 5 ++++-
  1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/base/daemon.c b/src/base/daemon.c
index 2f7f37f..e24eaaa 100644
--- a/src/base/daemon.c
+++ b/src/base/daemon.c
@@ -539,6 +539,9 @@ static void sigterm_handler(int signum, siginfo_t *info, 
void *ptr)
   */
  static void sighup_handler(int signum, siginfo_t *info, void *ptr)
  {
+       (void)signum;
+       (void)info;
+       (void)ptr;
        ncs_sel_obj_ind(&hangup_sel_obj);
  }
@@ -605,7 +608,7 @@ NCS_SEL_OBJ* daemon_sighup_install(int *hangup_fd) sigemptyset(&act.sa_mask);
        act.sa_sigaction = sighup_handler;
-       act.sa_flags = SA_SIGINFO;
+       act.sa_flags = SA_RESTART | SA_SIGINFO;
        if (sigaction(SIGHUP, &act, NULL) < 0) {
                syslog(LOG_ERR, "sigaction HUP failed: %s", strerror(errno));
                exit(EXIT_FAILURE);


_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to