better #219 fix.

git-svn-id: file:///svn/unbound/trunk@1386 be551aaa-1e26-0410-a405-d3ace91eadb9
This commit is contained in:
Wouter Wijngaards 2008-12-11 08:14:22 +00:00
parent b616c4f833
commit 05643589cf
3 changed files with 7 additions and 3 deletions

View file

@ -345,8 +345,6 @@ perform_setup(struct daemon* daemon, struct config_file* cfg, int debug_mode,
* a fork error could not be printed since daemonize closed stderr.*/
if(cfg->use_syslog) {
log_init(cfg->logfile, cfg->use_syslog, cfg->chrootdir);
/* but syslog is not really opened by glibc until first msg */
log_info("open syslog, startup in progress");
}
/* if using a logfile, we cannot open it because the logfile would
* be created with the wrong permissions, we cannot chown it because

View file

@ -1,3 +1,7 @@
11 December 2008: Wouter
- better fix for bug #219: use LOG_NDELAY with openlog() call.
Thanks to Tamas Tevesz.
9 December 2008: Wouter
- bug #221 fixed: unbound checkconf checks if key files exist if
remote control is enabled. Also fixed NULL printf when not chrooted.

View file

@ -95,7 +95,9 @@ log_init(const char* filename, int use_syslog, const char* chrootdir)
logging_to_syslog = 0;
}
if(use_syslog) {
openlog(ident, 0, LOG_DAEMON);
/* do not delay opening until first write, because we may
* chroot and no longer be able to access dev/log and so on */
openlog(ident, LOG_NDELAY, LOG_DAEMON);
logging_to_syslog = 1;
return;
}