Add warning comments to SIGHUP handlers, that we ignore the retrun code of reload_config().
This can actually fail, and we probably need some way to handle this. Like an emergency error exit or so... Michael
This commit is contained in:
parent
8346a20aa0
commit
0466526b1d
@ -167,6 +167,10 @@ short int child_configure (child_config_t type, unsigned int val)
|
||||
static void child_sighup_handler (int sig)
|
||||
{
|
||||
if (sig == SIGHUP) {
|
||||
/*
|
||||
* Ignore the return value of reload_config for now.
|
||||
* This should actually be handled somehow...
|
||||
*/
|
||||
reload_config ();
|
||||
|
||||
#ifdef FILTER_ENABLE
|
||||
@ -429,6 +433,10 @@ void child_main_loop (void)
|
||||
|
||||
/* Handle log rotation if it was requested */
|
||||
if (received_sighup) {
|
||||
/*
|
||||
* Ignore the return value of reload_config for now.
|
||||
* This should actually be handled somehow...
|
||||
*/
|
||||
reload_config ();
|
||||
|
||||
#ifdef FILTER_ENABLE
|
||||
|
Loading…
Reference in New Issue
Block a user