changeset 9894:3d362066d0e8 HEAD

master: Log startup errors as well as writing them to stderr.
author Timo Sirainen <tss@iki.fi>
date Sun, 06 Sep 2009 22:28:14 -0400
parents 906dbaba14a9
children 7681ae18ed03
files src/master/main.c
diffstat 1 files changed, 42 insertions(+), 12 deletions(-) [+]
line wrap: on
line diff
--- a/src/master/main.c	Sun Sep 06 21:47:35 2009 -0400
+++ b/src/master/main.c	Sun Sep 06 22:28:14 2009 -0400
@@ -42,6 +42,7 @@
 
 static char *pidfile_path;
 static fatal_failure_callback_t *orig_fatal_callback;
+static failure_callback_t *orig_error_callback;
 static const char *child_process_env[3]; /* @UNSAFE */
 
 static const struct setting_parser_info *set_roots[] = {
@@ -147,6 +148,28 @@
 	abort(); /* just to silence the noreturn attribute warnings */
 }
 
+static void ATTR_NORETURN
+startup_fatal_handler(enum log_type type, int status,
+		      const char *fmt, va_list args)
+{
+	fputs("Fatal: ", stderr);
+	vfprintf(stderr, fmt, args);
+	fputc('\n', stderr);
+
+	orig_fatal_callback(type, status, fmt, args);
+	abort();
+}
+
+static void
+startup_error_handler(enum log_type type, const char *fmt, va_list args)
+{
+	fputs("Error: ", stderr);
+	vfprintf(stderr, fmt, args);
+	fputc('\n', stderr);
+
+	orig_error_callback(type, fmt, args);
+}
+
 static void fatal_log_check(const struct master_settings *set)
 {
 	const char *path;
@@ -590,7 +613,7 @@
 	struct master_settings *set;
 	unsigned int child_process_env_idx = 0;
 	const char *getopt_str, *error, *env_tz, *doveconf_arg = NULL;
-	failure_callback_t *error_callback;
+	failure_callback_t *orig_info_callback;
 	void **sets;
 	bool foreground = FALSE, ask_key_pass = FALSE, log_error = FALSE;
 	int c, send_signal = 0;
@@ -695,6 +718,13 @@
 		i_strconcat(set->base_dir, "/"MASTER_PID_FILE_NAME, NULL);
 	if (send_signal != 0)
 		send_master_signal(send_signal);
+
+	master_service_init_log(master_service, "dovecot: ", 0);
+	i_get_failure_handlers(&orig_fatal_callback, &orig_error_callback,
+			       &orig_info_callback);
+	i_set_fatal_handler(startup_fatal_handler);
+	i_set_error_handler(startup_error_handler);
+
 	if (!log_error) {
 		pid_file_check_running(pidfile_path);
 		master_settings_do_fixes(set);
@@ -739,19 +769,19 @@
 
 	services->config->config_file_path = get_full_config_path(services);
 
-	/* if any listening fails, fail completely */
-	if (services_listen(services) <= 0)
-		return FATAL_DEFAULT;
+	if (!log_error) {
+		/* if any listening fails, fail completely */
+		if (services_listen(services) <= 0)
+			i_fatal("Failed to start listeners");
 
-	master_service_init_log(master_service, "dovecot: ", 0);
-	i_get_failure_handlers(&orig_fatal_callback, &error_callback,
-			       &error_callback);
-	i_set_fatal_handler(master_fatal_callback);
+		if (!foreground)
+			daemonize();
+		if (chdir(set->base_dir) < 0)
+			i_fatal("chdir(%s) failed: %m", set->base_dir);
+	}
 
-	if (!foreground)
-		daemonize();
-	if (chdir(set->base_dir) < 0)
-		i_fatal("chdir(%s) failed: %m", set->base_dir);
+	i_set_fatal_handler(master_fatal_callback);
+	i_set_error_handler(orig_error_callback);
 
 	main_init(log_error);
 	master_service_run(master_service, NULL);