diff options
| author | Denis Vlasenko | 2007-02-14 20:51:46 +0000 |
|---|---|---|
| committer | Denis Vlasenko | 2007-02-14 20:51:46 +0000 |
| commit | 4998c818919477d90674a3f220e7407c26da17f9 (patch) | |
| tree | b774ded50a81e7af4e1abfeb91abcbd4e0adefb6 /sysklogd | |
| parent | 9ab265883693cc8ff682ef44c3be7e49e00e4c97 (diff) | |
| download | busybox-4998c818919477d90674a3f220e7407c26da17f9.zip busybox-4998c818919477d90674a3f220e7407c26da17f9.tar.gz | |
syslogd: MARK code is buggy. Disabling it altogether on anti-bloat grounds
Diffstat (limited to 'sysklogd')
| -rw-r--r-- | sysklogd/syslogd.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c index 53290f1..53cb4ba 100644 --- a/sysklogd/syslogd.c +++ b/sysklogd/syslogd.c @@ -28,8 +28,13 @@ static const char *logFilePath = "/var/log/messages"; static int logFD = -1; +/* This is not very useful, is bloat, and broken: + * can deadlock if alarmed to make MARK while writing to IPC buffer + * (semaphores are down but do_mark routine tries to down them again) */ +#ifdef SYSLOGD_MARK /* interval between marks in seconds */ static int markInterval = 20 * 60; +#endif /* level of messages to be locally logged */ static int logLevel = 8; @@ -448,6 +453,7 @@ static void quit_signal(int sig) exit(1); } +#ifdef SYSLOGD_MARK static void do_mark(int sig) { if (markInterval) { @@ -455,6 +461,7 @@ static void do_mark(int sig) alarm(markInterval); } } +#endif static void do_syslogd(void) ATTRIBUTE_NORETURN; static void do_syslogd(void) @@ -473,8 +480,10 @@ static void do_syslogd(void) #ifdef SIGCLD signal(SIGCLD, SIG_IGN); #endif +#ifdef SYSLOGD_MARK signal(SIGALRM, do_mark); alarm(markInterval); +#endif memset(&sunx, 0, sizeof(sunx)); sunx.sun_family = AF_UNIX; @@ -558,8 +567,10 @@ int syslogd_main(int argc, char **argv) /* do normal option parsing */ opt_complementary = "=0"; /* no non-option params */ getopt32(argc, argv, OPTION_STR, OPTION_PARAM); +#ifdef SYSLOGD_MARK if (option_mask32 & OPT_mark) // -m markInterval = xatou_range(opt_m, 0, INT_MAX/60) * 60; +#endif //if (option_mask32 & OPT_nofork) // -n //if (option_mask32 & OPT_outfile) // -O if (option_mask32 & OPT_loglevel) // -l |