diff options
author | Jacek Masiulaniec <jacekm@cvs.openbsd.org> | 2009-03-29 14:18:21 +0000 |
---|---|---|
committer | Jacek Masiulaniec <jacekm@cvs.openbsd.org> | 2009-03-29 14:18:21 +0000 |
commit | 411f09d88bf2ce87a1b0767340ec1bffaa0444c5 (patch) | |
tree | de28bcbe59ddbac3ec4e91ba0f5a3fff026a7abb | |
parent | 2d278b45fcc694aac4935bcaddf4197306fcd0b1 (diff) |
turn some log_debugs into log_warns or even fatals; "looks ok" gilles@
-rw-r--r-- | usr.sbin/smtpd/control.c | 26 | ||||
-rw-r--r-- | usr.sbin/smtpd/lka.c | 28 | ||||
-rw-r--r-- | usr.sbin/smtpd/mda.c | 14 | ||||
-rw-r--r-- | usr.sbin/smtpd/mfa.c | 18 | ||||
-rw-r--r-- | usr.sbin/smtpd/mta.c | 18 | ||||
-rw-r--r-- | usr.sbin/smtpd/queue.c | 34 | ||||
-rw-r--r-- | usr.sbin/smtpd/runner.c | 22 | ||||
-rw-r--r-- | usr.sbin/smtpd/smtp.c | 22 | ||||
-rw-r--r-- | usr.sbin/smtpd/smtpd.c | 29 |
9 files changed, 103 insertions, 108 deletions
diff --git a/usr.sbin/smtpd/control.c b/usr.sbin/smtpd/control.c index 710ce3f27a5..ce1ce84ab93 100644 --- a/usr.sbin/smtpd/control.c +++ b/usr.sbin/smtpd/control.c @@ -1,4 +1,4 @@ -/* $OpenBSD: control.c,v 1.20 2009/03/18 14:48:27 gilles Exp $ */ +/* $OpenBSD: control.c,v 1.21 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Pierre-Yves Ritschard <pyr@openbsd.org> @@ -565,9 +565,9 @@ control_dispatch_parent(int sig, short event, void *p) break; } default: - log_debug("control_dispatch_parent: unexpected imsg %d", + log_warnx("control_dispatch_parent: got imsg %d", imsg.hdr.type); - break; + fatalx("control_dispatch_parent: unexpected imsg"); } imsg_free(&imsg); } @@ -621,9 +621,9 @@ control_dispatch_lka(int sig, short event, void *p) break; } default: - log_debug("control_dispatch_lka: unexpected imsg %d", + log_warnx("control_dispatch_lka: got imsg %d", imsg.hdr.type); - break; + fatalx("control_dispatch_lka: unexpected imsg"); } imsg_free(&imsg); } @@ -687,9 +687,9 @@ control_dispatch_mfa(int sig, short event, void *p) break; } default: - log_debug("control_dispatch_mfa: unexpected imsg %d", + log_warnx("control_dispatch_mfa: got imsg %d", imsg.hdr.type); - break; + fatalx("control_dispatch_mfa: unexpected imsg"); } imsg_free(&imsg); } @@ -836,9 +836,9 @@ control_dispatch_queue(int sig, short event, void *p) break; } default: - log_debug("control_dispatch_queue: unexpected imsg %d", + log_warnx("control_dispatch_queue: got imsg %d", imsg.hdr.type); - break; + fatalx("control_dispatch_queue: unexpected imsg"); } imsg_free(&imsg); } @@ -913,9 +913,9 @@ control_dispatch_runner(int sig, short event, void *p) break; } default: - log_debug("control_dispatch_runner: unexpected imsg %d", + log_warnx("control_dispatch_runner: got imsg %d", imsg.hdr.type); - break; + fatalx("control_dispatch_runner: unexpected imsg"); } imsg_free(&imsg); } @@ -974,9 +974,9 @@ control_dispatch_smtp(int sig, short event, void *p) break; } default: - log_debug("control_dispatch_smtp: unexpected imsg %d", + log_warnx("control_dispatch_smtp: got imsg %d", imsg.hdr.type); - break; + fatalx("control_dispatch_smtp: unexpected imsg"); } imsg_free(&imsg); } diff --git a/usr.sbin/smtpd/lka.c b/usr.sbin/smtpd/lka.c index 19c335f24e6..31ce61d6acf 100644 --- a/usr.sbin/smtpd/lka.c +++ b/usr.sbin/smtpd/lka.c @@ -1,4 +1,4 @@ -/* $OpenBSD: lka.c,v 1.37 2009/03/22 22:53:47 gilles Exp $ */ +/* $OpenBSD: lka.c,v 1.38 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Pierre-Yves Ritschard <pyr@openbsd.org> @@ -176,9 +176,9 @@ lka_dispatch_parent(int sig, short event, void *p) break; } default: - log_debug("parent_dispatch_lka: unexpected imsg %d", + log_warnx("lka_dispatch_parent: got imsg %d", imsg.hdr.type); - break; + fatalx("lka_dispatch_parent: unexpected imsg"); } imsg_free(&imsg); } @@ -312,9 +312,9 @@ lka_dispatch_mfa(int sig, short event, void *p) break; } default: - log_debug("lka_dispatch_mfa: unexpected imsg %d", + log_warnx("lka_dispatch_mfa: got imsg %d", imsg.hdr.type); - break; + fatalx("lka_dispatch_mfa: unexpected imsg"); } imsg_free(&imsg); } @@ -525,9 +525,9 @@ lka_dispatch_mta(int sig, short event, void *p) } default: - log_debug("lka_dispatch_mta: unexpected imsg %d", + log_warnx("lka_dispatch_mta: got imsg %d", imsg.hdr.type); - break; + fatalx("lka_dispatch_mta: unexpected imsg"); } imsg_free(&imsg); } @@ -597,9 +597,9 @@ lka_dispatch_smtp(int sig, short event, void *p) break; } default: - log_debug("lka_dispatch_mfa: unexpected imsg %d", + log_warnx("lka_dispatch_smtp: got imsg %d", imsg.hdr.type); - break; + fatalx("lka_dispatch_smtp: unexpected imsg"); } imsg_free(&imsg); } @@ -643,9 +643,9 @@ lka_dispatch_queue(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("lka_dispatch_queue: unexpected imsg %d", - imsg.hdr.type); - break; + log_warnx("lka_dispatch_queue: got imsg %d", + imsg.hdr.type); + fatalx("lka_dispatch_queue: unexpected imsg"); } imsg_free(&imsg); } @@ -689,9 +689,9 @@ lka_dispatch_runner(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("lka_dispatch_runner: unexpected imsg %d", + log_warnx("lka_dispatch_runner: got imsg %d", imsg.hdr.type); - break; + fatalx("lka_dispatch_runner: unexpected imsg"); } imsg_free(&imsg); } diff --git a/usr.sbin/smtpd/mda.c b/usr.sbin/smtpd/mda.c index 31609d4a852..a7707950632 100644 --- a/usr.sbin/smtpd/mda.c +++ b/usr.sbin/smtpd/mda.c @@ -1,4 +1,4 @@ -/* $OpenBSD: mda.c,v 1.12 2009/03/23 08:25:15 tobias Exp $ */ +/* $OpenBSD: mda.c,v 1.13 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Gilles Chehade <gilles@openbsd.org> @@ -175,9 +175,9 @@ mda_dispatch_parent(int sig, short event, void *p) break; } default: - log_debug("mda_dispatch_parent: unexpected imsg %d", + log_warnx("mda_dispatch_parent: got imsg %d", imsg.hdr.type); - break; + fatalx("mda_dispatch_parent: unexpected imsg"); } imsg_free(&imsg); } @@ -221,9 +221,9 @@ mda_dispatch_queue(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("parent_dispatch_queue: unexpected imsg %d", + log_warnx("mda_dispatch_queue: got imsg %d", imsg.hdr.type); - break; + fatalx("mda_dispatch_queue: unexpected imsg"); } imsg_free(&imsg); } @@ -345,9 +345,9 @@ mda_dispatch_runner(int sig, short event, void *p) break; } default: - log_debug("parent_dispatch_runner: unexpected imsg %d", + log_warnx("mda_dispatch_runner: got imsg %d", imsg.hdr.type); - break; + fatalx("mda_dispatch_runner: unexpected imsg"); } imsg_free(&imsg); } diff --git a/usr.sbin/smtpd/mfa.c b/usr.sbin/smtpd/mfa.c index 03081ea9c0d..d9b6cded642 100644 --- a/usr.sbin/smtpd/mfa.c +++ b/usr.sbin/smtpd/mfa.c @@ -1,4 +1,4 @@ -/* $OpenBSD: mfa.c,v 1.17 2009/03/08 19:11:22 gilles Exp $ */ +/* $OpenBSD: mfa.c,v 1.18 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Gilles Chehade <gilles@openbsd.org> @@ -105,9 +105,9 @@ mfa_dispatch_parent(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("parent_dispatch_mfa: unexpected imsg %d", + log_warnx("mfa_dispatch_parent: got imsg %d", imsg.hdr.type); - break; + fatalx("mfa_dispatch_parent: unexpected imsg"); } imsg_free(&imsg); } @@ -157,9 +157,9 @@ mfa_dispatch_smtp(int sig, short event, void *p) mfa_test_rcpt(env, imsg.data, PROC_SMTP); break; default: - log_debug("mfa_dispatch_smtp: unexpected imsg %d", + log_warnx("mfa_dispatch_smtp: got imsg %d", imsg.hdr.type); - break; + fatalx("mfa_dispatch_smtp: unexpected imsg"); } imsg_free(&imsg); } @@ -223,9 +223,9 @@ mfa_dispatch_lka(int sig, short event, void *p) break; } default: - log_debug("mfa_dispatch_lka: unexpected imsg %d", + log_warnx("mfa_dispatch_lka: got imsg %d", imsg.hdr.type); - break; + fatalx("mfa_dispatch_lka: unexpected imsg"); } imsg_free(&imsg); } @@ -272,9 +272,9 @@ mfa_dispatch_control(int sig, short event, void *p) mfa_test_rcpt(env, imsg.data, PROC_CONTROL); break; default: - log_debug("mfa_dispatch_smtp: unexpected imsg %d", + log_warnx("mfa_dispatch_control: got imsg %d", imsg.hdr.type); - break; + fatalx("mfa_dispatch_control: unexpected imsg"); } imsg_free(&imsg); } diff --git a/usr.sbin/smtpd/mta.c b/usr.sbin/smtpd/mta.c index 56dbf706dbb..7d9ba336cdb 100644 --- a/usr.sbin/smtpd/mta.c +++ b/usr.sbin/smtpd/mta.c @@ -1,4 +1,4 @@ -/* $OpenBSD: mta.c,v 1.38 2009/03/19 00:12:32 gilles Exp $ */ +/* $OpenBSD: mta.c,v 1.39 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Pierre-Yves Ritschard <pyr@openbsd.org> @@ -108,9 +108,9 @@ mta_dispatch_parent(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("parent_dispatch_mta: unexpected imsg %d", + log_warnx("mta_dispatch_parent: got imsg %d", imsg.hdr.type); - break; + fatalx("mta_dispatch_parent: unexpected imsg"); } imsg_free(&imsg); } @@ -202,9 +202,9 @@ mta_dispatch_lka(int sig, short event, void *p) break; } default: - log_debug("mta_dispatch_lka: unexpected imsg %d", + log_warnx("mta_dispatch_parent: got imsg %d", imsg.hdr.type); - break; + fatalx("mta_dispatch_lka: unexpected imsg"); } imsg_free(&imsg); } @@ -269,9 +269,9 @@ mta_dispatch_queue(int sig, short event, void *p) break; } default: - log_debug("parent_dispatch_mta: unexpected imsg %d", + log_warnx("mta_dispatch_queue: got imsg %d", imsg.hdr.type); - break; + fatalx("mta_dispatch_queue: unexpected imsg"); } imsg_free(&imsg); } @@ -389,9 +389,9 @@ mta_dispatch_runner(int sig, short event, void *p) break; } default: - log_debug("mta_dispatch_runner: unexpected imsg %d", + log_warnx("mta_dispatch_runner: got imsg %d", imsg.hdr.type); - break; + fatalx("mta_dispatch_runner: unexpected imsg"); } imsg_free(&imsg); } diff --git a/usr.sbin/smtpd/queue.c b/usr.sbin/smtpd/queue.c index d529eaf716c..278ee2207ab 100644 --- a/usr.sbin/smtpd/queue.c +++ b/usr.sbin/smtpd/queue.c @@ -1,4 +1,4 @@ -/* $OpenBSD: queue.c,v 1.57 2009/03/01 12:12:58 jacekm Exp $ */ +/* $OpenBSD: queue.c,v 1.58 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Gilles Chehade <gilles@openbsd.org> @@ -111,7 +111,7 @@ queue_dispatch_control(int sig, short event, void *p) struct message *messagep; struct submit_status ss; - log_debug("mfa_dispatch_control: creating message file"); + log_debug("queue_dispatch_control: creating message file"); messagep = imsg.data; ss.id = messagep->session_id; @@ -171,9 +171,9 @@ queue_dispatch_control(int sig, short event, void *p) break; } default: - log_debug("queue_dispatch_control: unexpected imsg %d", + log_warnx("queue_dispatch_control: got imsg %d", imsg.hdr.type); - break; + fatalx("queue_dispatch_control: unexpected imsg"); } imsg_free(&imsg); } @@ -220,7 +220,7 @@ queue_dispatch_smtp(int sig, short event, void *p) struct message *messagep; struct submit_status ss; - log_debug("mfa_dispatch_smtp: creating message file"); + log_debug("queue_dispatch_smtp: creating message file"); messagep = imsg.data; ss.id = messagep->session_id; ss.code = 250; @@ -274,9 +274,9 @@ queue_dispatch_smtp(int sig, short event, void *p) break; } default: - log_debug("queue_dispatch_smtp: unexpected imsg %d", + log_warnx("queue_dispatch_smtp: got imsg %d", imsg.hdr.type); - break; + fatalx("queue_dispatch_smtp: unexpected imsg"); } imsg_free(&imsg); } @@ -327,9 +327,8 @@ queue_dispatch_mda(int sig, short event, void *p) } default: - log_debug("queue_dispatch_mda: unexpected imsg %d", - imsg.hdr.type); - break; + log_warnx("got imsg %d", imsg.hdr.type); + fatalx("queue_dispatch_mda: unexpected imsg"); } imsg_free(&imsg); } @@ -391,9 +390,8 @@ queue_dispatch_mta(int sig, short event, void *p) } default: - log_debug("queue_dispatch_mda: unexpected imsg %d", - imsg.hdr.type); - break; + log_warnx("got imsg %d", imsg.hdr.type); + fatalx("queue_dispatch_mda: unexpected imsg"); } imsg_free(&imsg); } @@ -493,9 +491,8 @@ queue_dispatch_lka(int sig, short event, void *p) } default: - log_debug("queue_dispatch_lka: unexpected imsg %d", - imsg.hdr.type); - break; + log_warnx("got imsg %d", imsg.hdr.type); + fatalx("queue_dispatch_lka: unexpected imsg"); } imsg_free(&imsg); } @@ -539,9 +536,8 @@ queue_dispatch_runner(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("queue_dispatch_runner: unexpected imsg %d", - imsg.hdr.type); - break; + log_warnx("got imsg %d", imsg.hdr.type); + fatalx("queue_dispatch_runner: unexpected imsg"); } imsg_free(&imsg); } diff --git a/usr.sbin/smtpd/runner.c b/usr.sbin/smtpd/runner.c index 4144c425d1d..92023e7a788 100644 --- a/usr.sbin/smtpd/runner.c +++ b/usr.sbin/smtpd/runner.c @@ -1,4 +1,4 @@ -/* $OpenBSD: runner.c,v 1.37 2009/03/22 22:53:47 gilles Exp $ */ +/* $OpenBSD: runner.c,v 1.38 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Gilles Chehade <gilles@openbsd.org> @@ -165,9 +165,9 @@ runner_dispatch_control(int sig, short event, void *p) break; } default: - log_debug("queue_dispatch_control: unexpected imsg %d", + log_warnx("runner_dispatch_control: got imsg %d", imsg.hdr.type); - break; + fatalx("runner_dispatch_control: unexpected imsg"); } imsg_free(&imsg); } @@ -216,9 +216,9 @@ runner_dispatch_queue(int sig, short event, void *p) break; } default: - log_debug("runner_dispatch_queue: unexpected imsg %d", + log_warnx("runner_dispatch_queue: got imsg %d", imsg.hdr.type); - break; + fatalx("runner_dispatch_queue: unexpected imsg"); } imsg_free(&imsg); } @@ -262,9 +262,9 @@ runner_dispatch_mda(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("runner_dispatch_mda: unexpected imsg %d", + log_warnx("runner_dispatch_mda: got imsg %d", imsg.hdr.type); - break; + fatalx("runner_dispatch_mda: unexpected imsg"); } imsg_free(&imsg); } @@ -309,9 +309,9 @@ runner_dispatch_mta(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("runner_dispatch_mta: unexpected imsg %d", + log_warnx("runner_dispatch_mta: got imsg %d", imsg.hdr.type); - break; + fatalx("runner_dispatch_mta: unexpected imsg"); } imsg_free(&imsg); } @@ -355,9 +355,9 @@ runner_dispatch_lka(int sig, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("runner_dispatch_lka: unexpected imsg %d", + log_warnx("runner_dispatch_lka: got imsg %d", imsg.hdr.type); - break; + fatalx("runner_dispatch_lka: unexpected imsg"); } imsg_free(&imsg); } diff --git a/usr.sbin/smtpd/smtp.c b/usr.sbin/smtpd/smtp.c index 66e77c7d7e1..3fbf2f33f96 100644 --- a/usr.sbin/smtpd/smtp.c +++ b/usr.sbin/smtpd/smtp.c @@ -1,4 +1,4 @@ -/* $OpenBSD: smtp.c,v 1.31 2009/03/15 19:32:10 gilles Exp $ */ +/* $OpenBSD: smtp.c,v 1.32 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Gilles Chehade <gilles@openbsd.org> @@ -192,9 +192,9 @@ smtp_dispatch_parent(int sig, short event, void *p) break; } default: - log_debug("parent_dispatch_smtp: unexpected imsg %d", + log_warnx("smtp_dispatch_parent: got imsg %d", imsg.hdr.type); - break; + fatalx("smtp_dispatch_parent: unexpected imsg"); } imsg_free(&imsg); } @@ -262,9 +262,9 @@ smtp_dispatch_mfa(int sig, short event, void *p) break; } default: - log_debug("smtp_dispatch_mfa: unexpected imsg %d", + log_warnx("smtp_dispatch_mfa: got imsg %d", imsg.hdr.type); - break; + fatalx("smtp_dispatch_mfa: unexpected imsg"); } imsg_free(&imsg); } @@ -329,9 +329,9 @@ smtp_dispatch_lka(int sig, short event, void *p) break; } default: - log_debug("smtp_dispatch_lka: unexpected imsg %d", + log_warnx("smtp_dispatch_lka: got imsg %d", imsg.hdr.type); - break; + fatalx("smtp_dispatch_lka: unexpected imsg"); } imsg_free(&imsg); } @@ -487,9 +487,9 @@ smtp_dispatch_queue(int sig, short event, void *p) break; } default: - log_debug("smtp_dispatch_queue: unexpected imsg %d", + log_warnx("smtp_dispatch_queue: got imsg %d", imsg.hdr.type); - break; + fatalx("smtp_dispatch_queue: unexpected imsg"); } imsg_free(&imsg); } @@ -547,9 +547,9 @@ smtp_dispatch_control(int sig, short event, void *p) break; } default: - log_debug("smtp_dispatch_control: unexpected imsg %d", + log_warnx("smtp_dispatch_control: got imsg %d", imsg.hdr.type); - break; + fatalx("smtp_dispatch_control: unexpected imsg"); } imsg_free(&imsg); } diff --git a/usr.sbin/smtpd/smtpd.c b/usr.sbin/smtpd/smtpd.c index 215e9998d49..7328e0a4707 100644 --- a/usr.sbin/smtpd/smtpd.c +++ b/usr.sbin/smtpd/smtpd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: smtpd.c,v 1.50 2009/03/22 22:53:47 gilles Exp $ */ +/* $OpenBSD: smtpd.c,v 1.51 2009/03/29 14:18:20 jacekm Exp $ */ /* * Copyright (c) 2008 Gilles Chehade <gilles@openbsd.org> @@ -209,9 +209,9 @@ parent_dispatch_lka(int fd, short event, void *p) break; } default: - log_debug("parent_dispatch_lka: unexpected imsg %d", + log_warnx("parent_dispatch_lka: got imsg %d", imsg.hdr.type); - break; + fatalx("parent_dispatch_lka: unexpected imsg"); } imsg_free(&imsg); } @@ -255,9 +255,9 @@ parent_dispatch_mfa(int fd, short event, void *p) switch (imsg.hdr.type) { default: - log_debug("parent_dispatch_lka: unexpected imsg %d", + log_warnx("parent_dispatch_mfa: got imsg %d", imsg.hdr.type); - break; + fatalx("parent_dispatch_mfa: unexpected imsg"); } imsg_free(&imsg); } @@ -399,9 +399,9 @@ parent_dispatch_mda(int fd, short event, void *p) break; } default: - log_debug("parent_dispatch_mda: unexpected imsg %d", + log_warnx("parent_dispatch_mfa: got imsg %d", imsg.hdr.type); - break; + fatalx("parent_dispatch_mda: unexpected imsg"); } imsg_free(&imsg); } @@ -473,9 +473,9 @@ parent_dispatch_smtp(int fd, short event, void *p) break; } default: - log_debug("parent_dispatch_smtp: unexpected imsg %d", + log_warnx("parent_dispatch_smtp: got imsg %d", imsg.hdr.type); - break; + fatalx("parent_dispatch_smtp: unexpected imsg"); } imsg_free(&imsg); } @@ -527,9 +527,9 @@ parent_dispatch_control(int sig, short event, void *p) break; } default: - log_debug("parent_dispatch_control: unexpected imsg %d", + log_warnx("parent_dispatch_control: got imsg %d", imsg.hdr.type); - break; + fatalx("parent_dispatch_control: unexpected imsg"); } imsg_free(&imsg); } @@ -579,7 +579,7 @@ parent_sig_handler(int sig, short event, void *p) lookup.pid = pid; mdaproc = SPLAY_FIND(mdaproctree, &env->mdaproc_queue, &lookup); if (mdaproc == NULL) - errx(1, "received SIGCHLD but no known child for that pid (#%d)", pid); + fatalx("unexpected SIGCHLD"); if (WIFEXITED(status) && !WIFSIGNALED(status)) { switch (WEXITSTATUS(status)) { @@ -590,11 +590,11 @@ parent_sig_handler(int sig, short event, void *p) log_debug("DEBUG: external mda reported temporary failure"); break; default: - log_debug("DEBUG: external mda reported permanent failure"); + log_warnx("external mda returned %d", WEXITSTATUS(status)); } } else { - log_debug("DEBUG: external mda process has terminated in a baaaad way"); + log_warnx("external mda terminated abnormally"); } SPLAY_REMOVE(mdaproctree, &env->mdaproc_queue, @@ -603,7 +603,6 @@ parent_sig_handler(int sig, short event, void *p) } } while (pid > 0 || (pid == -1 && errno == EINTR)); - /**/ break; default: fatalx("unexpected signal"); |