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