diff options
| author | Ihor Antonov <ihor@antonovs.family> | 2023-03-02 03:00:42 +0000 |
|---|---|---|
| committer | Kyle Evans <kevans@FreeBSD.org> | 2023-03-02 03:00:42 +0000 |
| commit | 259ed21d21f81b656f06447716d41c190576477e (patch) | |
| tree | d532e8e6043c2e17d3d4349440cbb59de29f1ccd | |
| parent | 91b921c7d4684599d1d983f77dcc3df90900adc0 (diff) | |
| download | src-259ed21d21f81b656f06447716d41c190576477e.tar.gz src-259ed21d21f81b656f06447716d41c190576477e.zip | |
daemon: initialize struct sigaction at declaration site
This improves readability by uncluttering the code
Reviewed by: kevans
Pull Request: https://github.com/freebsd/freebsd-src/pull/669
| -rw-r--r-- | usr.sbin/daemon/daemon.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/usr.sbin/daemon/daemon.c b/usr.sbin/daemon/daemon.c index 4fe749d9e5fa..0925d16a27a0 100644 --- a/usr.sbin/daemon/daemon.c +++ b/usr.sbin/daemon/daemon.c @@ -287,20 +287,19 @@ main(int argc, char *argv[]) * and syslog. */ if (pidfile || ppidfile || restart || outfd != -1 || dosyslog) { - struct sigaction act_term, act_chld, act_hup; + struct sigaction act_term = { 0 }; + struct sigaction act_chld = { 0 }; + struct sigaction act_hup = { 0 }; /* Avoid PID racing with SIGCHLD and SIGTERM. */ - memset(&act_term, 0, sizeof(act_term)); act_term.sa_handler = handle_term; sigemptyset(&act_term.sa_mask); sigaddset(&act_term.sa_mask, SIGCHLD); - memset(&act_chld, 0, sizeof(act_chld)); act_chld.sa_handler = handle_chld; sigemptyset(&act_chld.sa_mask); sigaddset(&act_chld.sa_mask, SIGTERM); - memset(&act_hup, 0, sizeof(act_hup)); act_hup.sa_handler = handle_hup; sigemptyset(&act_hup.sa_mask); |