dovecot/original-hg/dovecot-1.2: src/master/login-process.c annotate

annotate src/master/login-process.c @ 4631:f5d53a684856 HEAD

SIGHUP caused memory corruption

author	Timo Sirainen <tss@iki.fi>
date	Mon, 25 Sep 2006 15:07:12 +0300
parents	9dc62b7594de
children	35ecfba4ab39

rev	line source
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	1 /* Copyright (C) 2002 Timo Sirainen */
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	2
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	3 #include "common.h"
532 3b53dd1280c6 I/O buffers now use real blocking instead of setting up a sub-ioloop to Timo Sirainen <tss@iki.fi> parents: 410 diff changeset	4 #include "ioloop.h"
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	5 #include "network.h"
764 f57c52738f90 Renamed IBuffer and OBuffer to IStream and OStream which describes their Timo Sirainen <tss@iki.fi> parents: 747 diff changeset	6 #include "ostream.h"
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	7 #include "fdpass.h"
727 8dd8ebe6bcac We use close-on-exec flag now to make sure that master process closes the Timo Sirainen <tss@iki.fi> parents: 722 diff changeset	8 #include "fd-close-on-exec.h"
697 7814b29d0862 Created env_put() and env_clean() for a bit easier handling of environment Timo Sirainen <tss@iki.fi> parents: 666 diff changeset	9 #include "env-util.h"
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	10 #include "restrict-access.h"
722 0438621d25ff Added virtual memory size limits to processes. Default values are pretty Timo Sirainen <tss@iki.fi> parents: 699 diff changeset	11 #include "restrict-process-size.h"
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	12 #include "login-process.h"
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	13 #include "auth-process.h"
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	14 #include "mail-process.h"
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	15 #include "master-login-interface.h"
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	16 #include "log.h"
3901 0ae5f5b468b7 Put ssl-parameters file into login directory so it still can be accessed Timo Sirainen <tss@iki.fi> parents: 3889 diff changeset	17 #include "ssl-init.h"
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	18
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	19 #include <unistd.h>
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	20 #include <syslog.h>
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	21
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	22 struct login_process {
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	23 struct login_group *group;
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	24 struct login_process prev_prelogin, next_prelogin;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	25 int refcount;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	26
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	27 pid_t pid;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	28 int fd;
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	29 struct io *io;
fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	30 struct ostream *output;
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	31 enum master_login_state state;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	32
925 2e649dec0f09 Auth and login processes send an "we're ok" reply at the end of Timo Sirainen <tss@iki.fi> parents: 903 diff changeset	33 unsigned int initialized:1;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	34 unsigned int destroyed:1;
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	35 unsigned int inetd_child:1;
613 1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	36 };
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	37
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	38 struct login_auth_request {
fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	39 struct login_process *process;
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	40 unsigned int tag;
fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	41
fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	42 unsigned int login_tag;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	43 int fd;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	44
2097 4e77cb0aff21 Added %l, %r and %P variables and mail_log_prefix setting. Timo Sirainen <tss@iki.fi> parents: 2046 diff changeset	45 struct ip_addr local_ip, remote_ip;
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	46 };
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	47
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	48 static unsigned int auth_id_counter, login_pid_counter;
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	49 static struct timeout *to;
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	50 static struct io *io_listen;
4148 f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	51 static bool logins_stalled = FALSE;
613 1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	52
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	53 static struct hash_table *processes;
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	54 static struct login_group *login_groups;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	55
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	56 static void login_process_destroy(struct login_process *p);
fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	57 static void login_process_unref(struct login_process *p);
3863 55df57c028d4 Added "bool" type and changed all ints that were used as booleans to bool. Timo Sirainen <tss@iki.fi> parents: 3692 diff changeset	58 static bool login_process_init_group(struct login_process *p);
4148 f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	59 static void login_processes_start_missing(void *context);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	60
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	61 static void login_group_create(struct settings *set)
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	62 {
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	63 struct login_group *group;
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	64
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	65 group = i_new(struct login_group, 1);
4631 f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	66 group->refcount = 1;
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	67 group->set = set;
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	68 group->process_type = set->protocol == MAIL_PROTOCOL_IMAP ?
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	69 PROCESS_TYPE_IMAP : PROCESS_TYPE_POP3;
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	70
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	71 group->next = login_groups;
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	72 login_groups = group;
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	73 }
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	74
4631 f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	75 static void login_group_unref(struct login_group *group)
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	76 {
4631 f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	77 i_assert(group->refcount > 0);
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	78
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	79 if (--group->refcount > 0)
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	80 return;
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	81
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	82 i_free(group);
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	83 }
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	84
2736 0f31778d3c34 Changed dovecot-auth protocol to ASCII based. Should be easier now to write Timo Sirainen <tss@iki.fi> parents: 2691 diff changeset	85 void auth_master_callback(const char user, const char const *args,
0f31778d3c34 Changed dovecot-auth protocol to ASCII based. Should be easier now to write Timo Sirainen <tss@iki.fi> parents: 2691 diff changeset	86 void *context)
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	87 {
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	88 struct login_auth_request *request = context;
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	89 struct master_login_reply master_reply;
2329 f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	90 ssize_t ret;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	91
2736 0f31778d3c34 Changed dovecot-auth protocol to ASCII based. Should be easier now to write Timo Sirainen <tss@iki.fi> parents: 2691 diff changeset	92 if (user == NULL)
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	93 master_reply.success = FALSE;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	94 else {
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	95 struct login_group *group = request->process->group;
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	96
2097 4e77cb0aff21 Added %l, %r and %P variables and mail_log_prefix setting. Timo Sirainen <tss@iki.fi> parents: 2046 diff changeset	97 t_push();
1039 793f05a7e50e signed/unsigned/const pointer fixes Timo Sirainen <tss@iki.fi> parents: 1036 diff changeset	98 master_reply.success =
4197 c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	99 create_mail_process(group->process_type, group->set,
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	100 request->fd, &request->local_ip,
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	101 &request->remote_ip, user, args,
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	102 FALSE);
2097 4e77cb0aff21 Added %l, %r and %P variables and mail_log_prefix setting. Timo Sirainen <tss@iki.fi> parents: 2046 diff changeset	103 t_pop();
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	104 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	105
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	106 /* reply to login */
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	107 master_reply.tag = request->login_tag;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	108
2329 f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	109 ret = o_stream_send(request->process->output, &master_reply,
f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	110 sizeof(master_reply));
f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	111 if (ret != sizeof(master_reply)) {
f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	112 if (ret >= 0) {
f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	113 i_warning("Login process %s transmit buffer full, "
f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	114 "killing..", dec2str(request->process->pid));
f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	115 }
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	116 login_process_destroy(request->process);
2329 f34aca0d7c85 Changed default cache fields. Timo Sirainen <tss@iki.fi> parents: 2141 diff changeset	117 }
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	118
805 5ac361acb316 Marked all non-trivial buffer modifications with @UNSAFE tag. Several Timo Sirainen <tss@iki.fi> parents: 801 diff changeset	119 if (close(request->fd) < 0)
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	120 i_error("close(mail client) failed: %m");
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	121 login_process_unref(request->process);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	122 i_free(request);
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	123 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	124
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	125 static void process_remove_from_prelogin_lists(struct login_process *p)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	126 {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	127 if (p->state != LOGIN_STATE_FULL_PRELOGINS)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	128 return;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	129
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	130 if (p->prev_prelogin == NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	131 p->group->oldest_prelogin_process = p->next_prelogin;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	132 else
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	133 p->prev_prelogin->next_prelogin = p->next_prelogin;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	134
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	135 if (p->next_prelogin == NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	136 p->group->newest_prelogin_process = p->prev_prelogin;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	137 else
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	138 p->next_prelogin->prev_prelogin = p->prev_prelogin;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	139
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	140 p->prev_prelogin = p->next_prelogin = NULL;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	141 }
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	142
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	143 static void process_mark_nonlistening(struct login_process *p,
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	144 enum master_login_state new_state)
614 e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	145 {
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	146 if (p->group == NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	147 return;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	148
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	149 if (p->state == LOGIN_STATE_LISTENING)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	150 p->group->listening_processes--;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	151
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	152 if (new_state == LOGIN_STATE_FULL_PRELOGINS) {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	153 /* add to prelogin list */
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	154 i_assert(p->state != new_state);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	155
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	156 p->prev_prelogin = p->group->newest_prelogin_process;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	157 if (p->group->newest_prelogin_process == NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	158 p->group->oldest_prelogin_process = p;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	159 else
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	160 p->group->newest_prelogin_process->next_prelogin = p;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	161 p->group->newest_prelogin_process = p;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	162 } else {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	163 process_remove_from_prelogin_lists(p);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	164 }
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	165 }
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	166
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	167 static void process_mark_listening(struct login_process *p)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	168 {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	169 if (p->group == NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	170 return;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	171
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	172 if (p->state != LOGIN_STATE_LISTENING)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	173 p->group->listening_processes++;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	174
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	175 process_remove_from_prelogin_lists(p);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	176 }
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	177
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	178 static void
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	179 login_process_set_state(struct login_process *p, enum master_login_state state)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	180 {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	181 if (state == p->state \|\| state > LOGIN_STATE_COUNT \|\|
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	182 (state < p->state && p->group->set->login_process_per_connection)) {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	183 i_error("login: tried to change state %d -> %d "
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	184 "(if you can't login at all, see src/lib/fdpass.c)",
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	185 p->state, state);
614 e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	186 return;
e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	187 }
e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	188
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	189 if (state == LOGIN_STATE_LISTENING) {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	190 process_mark_listening(p);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	191 } else {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	192 process_mark_nonlistening(p, state);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	193 }
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	194
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	195 p->state = state;
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	196 }
614 e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	197
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	198 static void login_process_groups_create(void)
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	199 {
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	200 struct server_settings *server;
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	201
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	202 for (server = settings_root; server != NULL; server = server->next) {
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	203 if (server->imap != NULL)
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	204 login_group_create(server->imap);
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	205 if (server->pop3 != NULL)
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	206 login_group_create(server->pop3);
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	207 }
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	208 }
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	209
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	210 static struct login_group *
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	211 login_group_process_find(const char *name, enum mail_protocol protocol)
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	212 {
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	213 struct login_group *group;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	214
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	215 if (login_groups == NULL)
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	216 login_process_groups_create();
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	217
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	218 for (group = login_groups; group != NULL; group = group->next) {
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	219 if (strcmp(group->set->server->name, name) == 0 &&
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	220 group->set->protocol == protocol)
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	221 return group;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	222 }
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	223
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	224 return NULL;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	225 }
614 e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	226
3863 55df57c028d4 Added "bool" type and changed all ints that were used as booleans to bool. Timo Sirainen <tss@iki.fi> parents: 3692 diff changeset	227 static bool login_process_read_group(struct login_process *p)
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	228 {
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	229 struct login_group *group;
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	230 const char name, proto;
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	231 char buf[256];
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	232 enum mail_protocol protocol;
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	233 unsigned int len;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	234 ssize_t ret;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	235
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	236 /* read length */
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	237 ret = read(p->fd, buf, 1);
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	238 if (ret != 1)
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	239 len = 0;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	240 else {
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	241 len = buf[0];
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	242 if (len >= sizeof(buf)) {
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	243 i_error("login: Server name length too large");
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	244 return FALSE;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	245 }
614 e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	246
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	247 ret = read(p->fd, buf, len);
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	248 }
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	249
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	250 if (ret < 0)
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	251 i_error("login: read() failed: %m");
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	252 else if (len == 0 \|\| (size_t)ret != len)
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	253 i_error("login: Server name wasn't sent");
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	254 else {
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	255 name = t_strndup(buf, len);
3607 a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	256 proto = strchr(name, '/');
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	257 if (proto == NULL) {
3607 a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	258 proto = name;
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	259 name = "default";
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	260 } else {
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	261 name = t_strdup_until(name, proto++);
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	262 }
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	263
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	264 if (strcmp(proto, "imap") == 0)
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	265 protocol = MAIL_PROTOCOL_IMAP;
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	266 else if (strcmp(proto, "pop3") == 0)
4353 3e542f308cb5 Starting pop3 processes from inetd started actually imap processes after Timo Sirainen <tss@iki.fi> parents: 4197 diff changeset	267 protocol = MAIL_PROTOCOL_POP3;
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	268 else {
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	269 i_error("login: Unknown protocol '%s'", proto);
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	270 return FALSE;
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	271 }
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	272
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	273 group = login_group_process_find(name, protocol);
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	274 if (group == NULL) {
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	275 i_error("login: Unknown server name '%s'", name);
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	276 return FALSE;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	277 }
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	278
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	279 p->group = group;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	280 return login_process_init_group(p);
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	281 }
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	282 return FALSE;
614 e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	283 }
e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	284
1036 f782b3319553 Removed useless parameters from io_callback_t and timeout_callback_t. Timo Sirainen <tss@iki.fi> parents: 1035 diff changeset	285 static void login_process_input(void *context)
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	286 {
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	287 struct login_process *p = context;
fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	288 struct auth_process *auth_process;
fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	289 struct login_auth_request *authreq;
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	290 struct master_login_request req;
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	291 int client_fd;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	292 ssize_t ret;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	293
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	294 if (p->group == NULL) {
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	295 /* we want to read the group */
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	296 if (!login_process_read_group(p))
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	297 login_process_destroy(p);
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	298 return;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	299 }
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	300
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	301 ret = fd_read(p->fd, &req, sizeof(req), &client_fd);
2779 1fd55b4341b9 Added version to login <-> master communication as well. Timo Sirainen <tss@iki.fi> parents: 2736 diff changeset	302 if (ret >= (ssize_t)sizeof(req.version) &&
1fd55b4341b9 Added version to login <-> master communication as well. Timo Sirainen <tss@iki.fi> parents: 2736 diff changeset	303 req.version != MASTER_LOGIN_PROTOCOL_VERSION) {
1fd55b4341b9 Added version to login <-> master communication as well. Timo Sirainen <tss@iki.fi> parents: 2736 diff changeset	304 i_error("login: Protocol version mismatch "
1fd55b4341b9 Added version to login <-> master communication as well. Timo Sirainen <tss@iki.fi> parents: 2736 diff changeset	305 "(mixed old and new binaries?)");
1fd55b4341b9 Added version to login <-> master communication as well. Timo Sirainen <tss@iki.fi> parents: 2736 diff changeset	306 login_process_destroy(p);
1fd55b4341b9 Added version to login <-> master communication as well. Timo Sirainen <tss@iki.fi> parents: 2736 diff changeset	307 return;
1fd55b4341b9 Added version to login <-> master communication as well. Timo Sirainen <tss@iki.fi> parents: 2736 diff changeset	308 }
4631 f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	309 {
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	310 static int i = 0;
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	311 if (i++ > 1) {
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	312 ret = -1; errno = EINVAL;
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	313 }
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	314 }
2779 1fd55b4341b9 Added version to login <-> master communication as well. Timo Sirainen <tss@iki.fi> parents: 2736 diff changeset	315
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	316 if (ret != sizeof(req)) {
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	317 if (ret == 0) {
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	318 /* disconnected, ie. the login process died */
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	319 } else if (ret > 0) {
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	320 /* req wasn't fully read */
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	321 i_error("login: fd_read() couldn't read all req");
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	322 } else {
4630 9dc62b7594de If fd_read() returns EAGAIN, ignore it. Timo Sirainen <tss@iki.fi> parents: 4538 diff changeset	323 if (errno == EAGAIN)
9dc62b7594de If fd_read() returns EAGAIN, ignore it. Timo Sirainen <tss@iki.fi> parents: 4538 diff changeset	324 return;
9dc62b7594de If fd_read() returns EAGAIN, ignore it. Timo Sirainen <tss@iki.fi> parents: 4538 diff changeset	325
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	326 i_error("login: fd_read() failed: %m");
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	327 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	328
929 a763b3826762 Don't leak fds in case of errors. Timo Sirainen <tss@iki.fi> parents: 925 diff changeset	329 if (client_fd != -1) {
a763b3826762 Don't leak fds in case of errors. Timo Sirainen <tss@iki.fi> parents: 925 diff changeset	330 if (close(client_fd) < 0)
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	331 i_error("close(mail client) failed: %m");
929 a763b3826762 Don't leak fds in case of errors. Timo Sirainen <tss@iki.fi> parents: 925 diff changeset	332 }
a763b3826762 Don't leak fds in case of errors. Timo Sirainen <tss@iki.fi> parents: 925 diff changeset	333
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	334 login_process_destroy(p);
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	335 return;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	336 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	337
613 1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	338 if (client_fd == -1) {
925 2e649dec0f09 Auth and login processes send an "we're ok" reply at the end of Timo Sirainen <tss@iki.fi> parents: 903 diff changeset	339 /* just a notification that the login process */
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	340 enum master_login_state state = req.tag;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	341
925 2e649dec0f09 Auth and login processes send an "we're ok" reply at the end of Timo Sirainen <tss@iki.fi> parents: 903 diff changeset	342 if (!p->initialized) {
2e649dec0f09 Auth and login processes send an "we're ok" reply at the end of Timo Sirainen <tss@iki.fi> parents: 903 diff changeset	343 /* initialization notify */
2e649dec0f09 Auth and login processes send an "we're ok" reply at the end of Timo Sirainen <tss@iki.fi> parents: 903 diff changeset	344 p->initialized = TRUE;;
2e649dec0f09 Auth and login processes send an "we're ok" reply at the end of Timo Sirainen <tss@iki.fi> parents: 903 diff changeset	345 } else {
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	346 /* change "listening for new connections" status */
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	347 login_process_set_state(p, state);
925 2e649dec0f09 Auth and login processes send an "we're ok" reply at the end of Timo Sirainen <tss@iki.fi> parents: 903 diff changeset	348 }
613 1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	349 return;
1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	350 }
1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	351
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	352 fd_close_on_exec(client_fd, TRUE);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	353
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	354 /* ask the cookie from the auth process */
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	355 authreq = i_new(struct login_auth_request, 1);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	356 p->refcount++;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	357 authreq->process = p;
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	358 authreq->tag = ++auth_id_counter;
fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	359 authreq->login_tag = req.tag;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	360 authreq->fd = client_fd;
2097 4e77cb0aff21 Added %l, %r and %P variables and mail_log_prefix setting. Timo Sirainen <tss@iki.fi> parents: 2046 diff changeset	361 authreq->local_ip = req.local_ip;
4e77cb0aff21 Added %l, %r and %P variables and mail_log_prefix setting. Timo Sirainen <tss@iki.fi> parents: 2046 diff changeset	362 authreq->remote_ip = req.remote_ip;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	363
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	364 auth_process = auth_process_find(req.auth_pid);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	365 if (auth_process == NULL) {
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	366 i_error("login: Authentication process %u doesn't exist",
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	367 req.auth_pid);
fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	368 auth_master_callback(NULL, NULL, authreq);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	369 } else {
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	370 auth_process_request(auth_process, p->pid,
fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	371 req.auth_id, authreq);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	372 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	373 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	374
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	375 static struct login_process *
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	376 login_process_new(struct login_group *group, pid_t pid, int fd)
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	377 {
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	378 struct login_process *p;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	379
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	380 i_assert(pid != 0);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	381
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	382 p = i_new(struct login_process, 1);
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	383 p->group = group;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	384 p->refcount = 1;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	385 p->pid = pid;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	386 p->fd = fd;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	387 p->io = io_add(fd, IO_READ, login_process_input, p);
764 f57c52738f90 Renamed IBuffer and OBuffer to IStream and OStream which describes their Timo Sirainen <tss@iki.fi> parents: 747 diff changeset	388 p->output = o_stream_create_file(fd, default_pool,
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	389 sizeof(struct master_login_reply)*10,
1499 e850252cdc7e Removed I/O priorities. They were pretty much useless and were just getting Timo Sirainen <tss@iki.fi> parents: 1468 diff changeset	390 FALSE);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	391
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	392 PID_ADD_PROCESS_TYPE(pid, PROCESS_TYPE_LOGIN);
195 db6e288be0e9 Replaced INT_TO_POINTER and POINTER_TO_INT macros with POINTER_CAST and Timo Sirainen <tss@iki.fi> parents: 10 diff changeset	393 hash_insert(processes, POINTER_CAST(pid), p);
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	394
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	395 p->state = LOGIN_STATE_LISTENING;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	396
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	397 if (p->group != NULL) {
4631 f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	398 p->group->refcount++;
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	399 p->group->processes++;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	400 p->group->listening_processes++;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	401 }
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	402 return p;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	403 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	404
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	405 static void login_process_exited(struct login_process *p)
613 1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	406 {
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	407 if (p->group != NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	408 p->group->processes--;
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	409
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	410 hash_remove(processes, POINTER_CAST(p->pid));
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	411 login_process_unref(p);
613 1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	412 }
1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	413
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	414 static void login_process_destroy(struct login_process *p)
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	415 {
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	416 if (p->destroyed)
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	417 return;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	418 p->destroyed = TRUE;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	419
1035 fe49ece0f3ea We have now separate "userdb" and "passdb". They aren't tied to each others Timo Sirainen <tss@iki.fi> parents: 1000 diff changeset	420 if (!p->initialized && io_loop_is_running(ioloop)) {
1000 0fbafade2d85 If auth/login process died unexpectedly, the exit status or killing signal Timo Sirainen <tss@iki.fi> parents: 945 diff changeset	421 i_error("Login process died too early - shutting down");
0fbafade2d85 If auth/login process died unexpectedly, the exit status or killing signal Timo Sirainen <tss@iki.fi> parents: 945 diff changeset	422 io_loop_stop(ioloop);
0fbafade2d85 If auth/login process died unexpectedly, the exit status or killing signal Timo Sirainen <tss@iki.fi> parents: 945 diff changeset	423 }
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	424
764 f57c52738f90 Renamed IBuffer and OBuffer to IStream and OStream which describes their Timo Sirainen <tss@iki.fi> parents: 747 diff changeset	425 o_stream_close(p->output);
3879 928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	426 io_remove(&p->io);
805 5ac361acb316 Marked all non-trivial buffer modifications with @UNSAFE tag. Several Timo Sirainen <tss@iki.fi> parents: 801 diff changeset	427 if (close(p->fd) < 0)
5ac361acb316 Marked all non-trivial buffer modifications with @UNSAFE tag. Several Timo Sirainen <tss@iki.fi> parents: 801 diff changeset	428 i_error("close(login) failed: %m");
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	429
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	430 process_mark_nonlistening(p, LOGIN_STATE_FULL_LOGINS);
613 1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	431
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	432 if (p->inetd_child)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	433 login_process_exited(p);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	434 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	435
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	436 static void login_process_unref(struct login_process *p)
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	437 {
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	438 if (--p->refcount > 0)
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	439 return;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	440
4631 f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	441 if (p->group != NULL)
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	442 login_group_unref(p->group);
f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	443
3879 928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	444 o_stream_unref(&p->output);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	445 i_free(p);
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	446 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	447
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	448 static void login_process_init_env(struct login_group *group, pid_t pid)
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	449 {
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	450 struct settings *set = group->set;
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	451
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	452 child_process_init_env();
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	453
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	454 /* setup access environment - needs to be done after
3692 162ad2307710 Comment update Timo Sirainen <tss@iki.fi> parents: 3691 diff changeset	455 clean_child_process() since it clears environment. Don't set user
162ad2307710 Comment update Timo Sirainen <tss@iki.fi> parents: 3691 diff changeset	456 parameter since we don't want to call initgroups() for login
3691 e167b71e37e3 Don't call initgroups() at the start of login processes. Timo Sirainen <tss@iki.fi> parents: 3607 diff changeset	457 processes. */
e167b71e37e3 Don't call initgroups() at the start of login processes. Timo Sirainen <tss@iki.fi> parents: 3607 diff changeset	458 restrict_access_set_env(NULL, set->login_uid,
1724 b3526668de78 Make sure auth process and login process don't share uids. Timo Sirainen <tss@iki.fi> parents: 1610 diff changeset	459 set->server->login_gid,
1506 e7c627bacaaf Allow first_valid_gid to be 0. Drop any supplementary groups not in valid Timo Sirainen <tss@iki.fi> parents: 1499 diff changeset	460 set->login_chroot ? set->login_dir : NULL,
2141 8690d2000e33 Added mail_extra_groups setting. Timo Sirainen <tss@iki.fi> parents: 2097 diff changeset	461 0, 0, NULL);
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	462
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	463 env_put("DOVECOT_MASTER=1");
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	464
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	465 if (!set->ssl_disable) {
4388 af61031c746f Settings parser nowadays returns "" instead of NULL when it reads an empty Timo Sirainen <tss@iki.fi> parents: 4353 diff changeset	466 const char *ssl_key_password;
3889 c7462001227b Added support for password protected SSL private keys. The password can be Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	467
4388 af61031c746f Settings parser nowadays returns "" instead of NULL when it reads an empty Timo Sirainen <tss@iki.fi> parents: 4353 diff changeset	468 ssl_key_password = *set->ssl_key_password != '\0' ?
3889 c7462001227b Added support for password protected SSL private keys. The password can be Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	469 set->ssl_key_password : ssl_manual_key_password;
c7462001227b Added support for password protected SSL private keys. The password can be Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	470
4388 af61031c746f Settings parser nowadays returns "" instead of NULL when it reads an empty Timo Sirainen <tss@iki.fi> parents: 4353 diff changeset	471 if (*set->ssl_ca_file != '\0') {
1907 190f1d315ce6 Added setting ssl_ca_file, patch by Zach Bagnall Timo Sirainen <tss@iki.fi> parents: 1897 diff changeset	472 env_put(t_strconcat("SSL_CA_FILE=",
190f1d315ce6 Added setting ssl_ca_file, patch by Zach Bagnall Timo Sirainen <tss@iki.fi> parents: 1897 diff changeset	473 set->ssl_ca_file, NULL));
190f1d315ce6 Added setting ssl_ca_file, patch by Zach Bagnall Timo Sirainen <tss@iki.fi> parents: 1897 diff changeset	474 }
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	475 env_put(t_strconcat("SSL_CERT_FILE=",
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	476 set->ssl_cert_file, NULL));
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	477 env_put(t_strconcat("SSL_KEY_FILE=",
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	478 set->ssl_key_file, NULL));
3889 c7462001227b Added support for password protected SSL private keys. The password can be Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	479 env_put(t_strconcat("SSL_KEY_PASSWORD=",
c7462001227b Added support for password protected SSL private keys. The password can be Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	480 ssl_key_password, NULL));
3901 0ae5f5b468b7 Put ssl-parameters file into login directory so it still can be accessed Timo Sirainen <tss@iki.fi> parents: 3889 diff changeset	481 env_put("SSL_PARAM_FILE="SSL_PARAMETERS_FILENAME);
4388 af61031c746f Settings parser nowadays returns "" instead of NULL when it reads an empty Timo Sirainen <tss@iki.fi> parents: 4353 diff changeset	482 if (*set->ssl_cipher_list != '\0') {
1996 d8f06a0c818e Added ssl_cipher_list setting. Timo Sirainen <tss@iki.fi> parents: 1907 diff changeset	483 env_put(t_strconcat("SSL_CIPHER_LIST=",
d8f06a0c818e Added ssl_cipher_list setting. Timo Sirainen <tss@iki.fi> parents: 1907 diff changeset	484 set->ssl_cipher_list, NULL));
d8f06a0c818e Added ssl_cipher_list setting. Timo Sirainen <tss@iki.fi> parents: 1907 diff changeset	485 }
1997 1d0985f6bdd9 Added ssl_verify_client_cert setting. Timo Sirainen <tss@iki.fi> parents: 1996 diff changeset	486 if (set->ssl_verify_client_cert)
1d0985f6bdd9 Added ssl_verify_client_cert setting. Timo Sirainen <tss@iki.fi> parents: 1996 diff changeset	487 env_put("SSL_VERIFY_CLIENT_CERT=1");
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	488 }
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	489
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	490 if (set->disable_plaintext_auth)
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	491 env_put("DISABLE_PLAINTEXT_AUTH=1");
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	492 if (set->verbose_proctitle)
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	493 env_put("VERBOSE_PROCTITLE=1");
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	494 if (set->verbose_ssl)
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	495 env_put("VERBOSE_SSL=1");
2691 46f879c46b45 auth_verbose now affects imap/pop3 login processes too. Every authentication Timo Sirainen <tss@iki.fi> parents: 2674 diff changeset	496 if (set->server->auths->verbose)
46f879c46b45 auth_verbose now affects imap/pop3 login processes too. Every authentication Timo Sirainen <tss@iki.fi> parents: 2674 diff changeset	497 env_put("VERBOSE_AUTH=1");
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	498
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	499 if (set->login_process_per_connection) {
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	500 env_put("PROCESS_PER_CONNECTION=1");
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	501 env_put("MAX_LOGGING_USERS=1");
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	502 } else {
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	503 env_put(t_strdup_printf("MAX_CONNECTIONS=%u",
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	504 set->login_max_connections));
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	505 }
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	506
2674 857f5f7b512b Added login_greeting and login_greeting_capability settings. Timo Sirainen <tss@iki.fi> parents: 2329 diff changeset	507 env_put(t_strconcat("PROCESS_UID=", dec2str(pid), NULL));
857f5f7b512b Added login_greeting and login_greeting_capability settings. Timo Sirainen <tss@iki.fi> parents: 2329 diff changeset	508 env_put(t_strconcat("GREETING=", set->login_greeting, NULL));
3384 3b75956d20c4 Added configurable logging for login process. Added configurable pop3 logout Timo Sirainen <tss@iki.fi> parents: 3083 diff changeset	509 env_put(t_strconcat("LOG_FORMAT_ELEMENTS=",
3b75956d20c4 Added configurable logging for login process. Added configurable pop3 logout Timo Sirainen <tss@iki.fi> parents: 3083 diff changeset	510 set->login_log_format_elements, NULL));
3b75956d20c4 Added configurable logging for login process. Added configurable pop3 logout Timo Sirainen <tss@iki.fi> parents: 3083 diff changeset	511 env_put(t_strconcat("LOG_FORMAT=", set->login_log_format, NULL));
2674 857f5f7b512b Added login_greeting and login_greeting_capability settings. Timo Sirainen <tss@iki.fi> parents: 2329 diff changeset	512 if (set->login_greeting_capability)
857f5f7b512b Added login_greeting and login_greeting_capability settings. Timo Sirainen <tss@iki.fi> parents: 2329 diff changeset	513 env_put("GREETING_CAPABILITY=1");
4197 c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	514
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	515 if (group->process_type == PROCESS_TYPE_IMAP) {
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	516 env_put(t_strconcat("CAPABILITY_STRING=",
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	517 *set->imap_capability != '\0' ?
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	518 set->imap_capability :
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	519 set->imap_generated_capability, NULL));
c3ded5b815aa If we have plugins set and imap_capability unset, figure out the IMAP Timo Sirainen <tss@iki.fi> parents: 4148 diff changeset	520 }
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	521 }
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	522
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	523 static pid_t create_login_process(struct login_group *group)
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	524 {
2046 6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	525 struct log_io *log;
3083 a20882c4f092 Disable log throttlong for auth process, and for login processes if Timo Sirainen <tss@iki.fi> parents: 2886 diff changeset	526 unsigned int max_log_lines_per_sec;
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	527 const char *prefix;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	528 pid_t pid;
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	529 int fd[2], log_fd;
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	530
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	531 if (group->set->login_uid == 0)
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	532 i_fatal("Login process must not run as root");
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	533
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	534 /* create communication to process with a socket pair */
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	535 if (socketpair(AF_UNIX, SOCK_STREAM, 0, fd) < 0) {
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	536 i_error("socketpair() failed: %m");
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	537 return -1;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	538 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	539
3083 a20882c4f092 Disable log throttlong for auth process, and for login processes if Timo Sirainen <tss@iki.fi> parents: 2886 diff changeset	540 max_log_lines_per_sec =
a20882c4f092 Disable log throttlong for auth process, and for login processes if Timo Sirainen <tss@iki.fi> parents: 2886 diff changeset	541 group->set->login_process_per_connection ? 10 : 0;
a20882c4f092 Disable log throttlong for auth process, and for login processes if Timo Sirainen <tss@iki.fi> parents: 2886 diff changeset	542 log_fd = log_create_pipe(&log, max_log_lines_per_sec);
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	543 if (log_fd < 0)
c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	544 pid = -1;
c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	545 else {
c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	546 pid = fork();
c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	547 if (pid < 0)
c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	548 i_error("fork() failed: %m");
c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	549 }
c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	550
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	551 if (pid < 0) {
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	552 (void)close(fd[0]);
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	553 (void)close(fd[1]);
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	554 (void)close(log_fd);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	555 return -1;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	556 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	557
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	558 if (pid != 0) {
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	559 /* master */
2046 6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	560 prefix = t_strdup_printf("%s-login: ",
6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	561 process_names[group->process_type]);
6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	562 log_set_prefix(log, prefix);
6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	563
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	564 net_set_nonblock(fd[0], TRUE);
727 8dd8ebe6bcac We use close-on-exec flag now to make sure that master process closes the Timo Sirainen <tss@iki.fi> parents: 722 diff changeset	565 fd_close_on_exec(fd[0], TRUE);
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	566 (void)login_process_new(group, pid, fd[0]);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	567 (void)close(fd[1]);
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	568 (void)close(log_fd);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	569 return pid;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	570 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	571
2046 6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	572 prefix = t_strdup_printf("master-%s-login: ",
6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	573 process_names[group->process_type]);
6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	574 log_set_prefix(log, prefix);
6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	575
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	576 /* move the listen handle */
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	577 if (dup2(group->set->listen_fd, LOGIN_LISTEN_FD) < 0)
2046 6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	578 i_fatal("dup2(listen_fd) failed: %m");
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	579 fd_close_on_exec(LOGIN_LISTEN_FD, FALSE);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	580
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	581 /* move the SSL listen handle */
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	582 if (dup2(group->set->ssl_listen_fd, LOGIN_SSL_LISTEN_FD) < 0)
2046 6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	583 i_fatal("dup2(ssl_listen_fd) failed: %m");
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	584 fd_close_on_exec(LOGIN_SSL_LISTEN_FD, FALSE);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	585
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	586 /* move communication handle */
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	587 if (dup2(fd[1], LOGIN_MASTER_SOCKET_FD) < 0)
2046 6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	588 i_fatal("dup2(master) failed: %m");
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	589 fd_close_on_exec(LOGIN_MASTER_SOCKET_FD, FALSE);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	590
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	591 if (dup2(log_fd, 2) < 0)
2046 6ae973f60f43 logging cleanups. Timo Sirainen <tss@iki.fi> parents: 2000 diff changeset	592 i_fatal("dup2(stderr) failed: %m");
2000 c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	593 fd_close_on_exec(2, FALSE);
c7c19f5071c3 Write all logging through master process. Fixes problems with log rotation, Timo Sirainen <tss@iki.fi> parents: 1997 diff changeset	594
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	595 (void)close(fd[0]);
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	596 (void)close(fd[1]);
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	597
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	598 login_process_init_env(group, getpid());
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	599
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	600 if (!group->set->login_chroot) {
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	601 /* no chrooting, but still change to the directory */
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	602 if (chdir(group->set->login_dir) < 0) {
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	603 i_fatal("chdir(%s) failed: %m",
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	604 group->set->login_dir);
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	605 }
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	606 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	607
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	608 restrict_process_size(group->set->login_process_size, (unsigned int)-1);
722 0438621d25ff Added virtual memory size limits to processes. Default values are pretty Timo Sirainen <tss@iki.fi> parents: 699 diff changeset	609
801 86224ff16bf6 Drop root privileges earlier. Close syslog more later in imap-master when Timo Sirainen <tss@iki.fi> parents: 791 diff changeset	610 /* make sure we don't leak syslog fd, but do it last so that
86224ff16bf6 Drop root privileges earlier. Close syslog more later in imap-master when Timo Sirainen <tss@iki.fi> parents: 791 diff changeset	611 any errors above will be logged */
86224ff16bf6 Drop root privileges earlier. Close syslog more later in imap-master when Timo Sirainen <tss@iki.fi> parents: 791 diff changeset	612 closelog();
86224ff16bf6 Drop root privileges earlier. Close syslog more later in imap-master when Timo Sirainen <tss@iki.fi> parents: 791 diff changeset	613
2886 634cf6aa34d1 Allow running all executables with parameters. Timo Sirainen <tss@iki.fi> parents: 2779 diff changeset	614 client_process_exec(group->set->login_executable, "");
1441 a83ce8395316 If exec() fails, show the full path rather than argv[0] Timo Sirainen <tss@iki.fi> parents: 1398 diff changeset	615 i_fatal_status(FATAL_EXEC, "execv(%s) failed: %m",
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	616 group->set->login_executable);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	617 return -1;
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	618 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	619
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	620 void login_process_destroyed(pid_t pid, bool abnormal_exit)
615 0d852af6842e Master process generates DH/RSA parameters now and stores them into file Timo Sirainen <tss@iki.fi> parents: 614 diff changeset	621 {
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	622 struct login_process *p;
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	623
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	624 p = hash_lookup(processes, POINTER_CAST(pid));
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	625 if (p == NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	626 i_panic("Lost login process PID %s", dec2str(pid));
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	627 i_assert(!p->inetd_child);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	628
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	629 if (abnormal_exit) {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	630 /* don't start raising the process count if they're dying all
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	631 the time */
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	632 if (p->group != NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	633 p->group->wanted_processes_count = 0;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	634 }
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	635
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	636 login_process_destroy(p);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	637 login_process_exited(p);
615 0d852af6842e Master process generates DH/RSA parameters now and stores them into file Timo Sirainen <tss@iki.fi> parents: 614 diff changeset	638 }
0d852af6842e Master process generates DH/RSA parameters now and stores them into file Timo Sirainen <tss@iki.fi> parents: 614 diff changeset	639
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	640 void login_processes_destroy_all(bool unref)
635 90a65c017bf0 SIGHUP reloads now settings. Logged in clients are left untouched, but Timo Sirainen <tss@iki.fi> parents: 615 diff changeset	641 {
1897 1e6ed8045f2b Changed hash_foreach() to iterator. Timo Sirainen <tss@iki.fi> parents: 1724 diff changeset	642 struct hash_iterate_context *iter;
1e6ed8045f2b Changed hash_foreach() to iterator. Timo Sirainen <tss@iki.fi> parents: 1724 diff changeset	643 void key, value;
1e6ed8045f2b Changed hash_foreach() to iterator. Timo Sirainen <tss@iki.fi> parents: 1724 diff changeset	644
1e6ed8045f2b Changed hash_foreach() to iterator. Timo Sirainen <tss@iki.fi> parents: 1724 diff changeset	645 iter = hash_iterate_init(processes);
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	646 while (hash_iterate(iter, &key, &value)) {
1897 1e6ed8045f2b Changed hash_foreach() to iterator. Timo Sirainen <tss@iki.fi> parents: 1724 diff changeset	647 login_process_destroy(value);
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	648 if (unref) login_process_unref(value);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	649 }
1897 1e6ed8045f2b Changed hash_foreach() to iterator. Timo Sirainen <tss@iki.fi> parents: 1724 diff changeset	650 hash_iterate_deinit(iter);
635 90a65c017bf0 SIGHUP reloads now settings. Logged in clients are left untouched, but Timo Sirainen <tss@iki.fi> parents: 615 diff changeset	651
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	652 while (login_groups != NULL) {
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	653 struct login_group *group = login_groups;
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	654
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	655 login_groups = group->next;
4631 f5d53a684856 SIGHUP caused memory corruption Timo Sirainen <tss@iki.fi> parents: 4630 diff changeset	656 login_group_unref(group);
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	657 }
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	658 }
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	659
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	660 static void login_processes_notify_group(struct login_group *group)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	661 {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	662 struct hash_iterate_context *iter;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	663 struct master_login_reply reply;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	664 void key, value;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	665
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	666 memset(&reply, 0, sizeof(reply));
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	667
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	668 iter = hash_iterate_init(processes);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	669 while (hash_iterate(iter, &key, &value)) {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	670 struct login_process *p = value;
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	671
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	672 if (p->group == group)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	673 (void)o_stream_send(p->output, &reply, sizeof(reply));
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	674 }
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	675 hash_iterate_deinit(iter);
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	676 }
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	677
4148 f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	678 static int login_group_start_missings(struct login_group *group)
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	679 {
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	680 if (group->set->login_process_per_connection &&
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	681 group->processes >= group->set->login_max_processes_count &&
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	682 group->listening_processes == 0) {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	683 /* destroy the oldest listening process. non-listening
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	684 processes are logged in users who we don't want to kick out
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	685 because someone's started flooding */
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	686 if (group->oldest_prelogin_process != NULL)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	687 login_process_destroy(group->oldest_prelogin_process);
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	688 }
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	689
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	690 /* we want to respond fast when multiple clients are connecting
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	691 at once, but we also want to prevent fork-bombing. use the
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	692 same method as apache: check once a second if we need new
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	693 processes. if yes and we've used all the existing processes,
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	694 double their amount (unless we've hit the high limit).
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	695 Then for each second that didn't use all existing processes,
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	696 drop the max. process count by one. */
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	697 if (group->wanted_processes_count < group->set->login_processes_count) {
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	698 group->wanted_processes_count =
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	699 group->set->login_processes_count;
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	700 } else if (group->listening_processes == 0)
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	701 group->wanted_processes_count *= 2;
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	702 else if (group->wanted_processes_count >
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	703 group->set->login_processes_count)
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	704 group->wanted_processes_count--;
a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	705
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	706 while (group->listening_processes < group->wanted_processes_count &&
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	707 group->processes < group->set->login_max_processes_count) {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	708 if (create_login_process(group) < 0)
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	709 return -1;
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	710 }
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	711
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	712 if (group->listening_processes == 0 &&
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	713 !group->set->login_process_per_connection) {
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	714 /* we've reached our limit. notify the processes to start
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	715 listening again which makes them kill some of their
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	716 oldest clients when accepting the next connection */
9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	717 login_processes_notify_group(group);
4148 f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	718 }
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	719 return 0;
635 90a65c017bf0 SIGHUP reloads now settings. Logged in clients are left untouched, but Timo Sirainen <tss@iki.fi> parents: 615 diff changeset	720 }
90a65c017bf0 SIGHUP reloads now settings. Logged in clients are left untouched, but Timo Sirainen <tss@iki.fi> parents: 615 diff changeset	721
4148 f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	722 static void login_processes_stall(void)
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	723 {
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	724 if (logins_stalled)
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	725 return;
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	726
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	727 i_error("Temporary failure in creating login processes, "
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	728 "slowing down for now");
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	729 logins_stalled = TRUE;
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	730
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	731 timeout_remove(&to);
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	732 to = timeout_add(60*1000, login_processes_start_missing, NULL);
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	733 }
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	734
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	735
903 fd8888f6f037 Naming style changes, finally got tired of most of the typedefs. Also the Timo Sirainen <tss@iki.fi> parents: 876 diff changeset	736 static void
1036 f782b3319553 Removed useless parameters from io_callback_t and timeout_callback_t. Timo Sirainen <tss@iki.fi> parents: 1035 diff changeset	737 login_processes_start_missing(void *context __attr_unused__)
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	738 {
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	739 struct login_group *group;
614 e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	740
1610 6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	741 if (login_groups == NULL)
6850142c4e25 New configuration file code. Some syntax changes, but tries to be somewhat Timo Sirainen <tss@iki.fi> parents: 1506 diff changeset	742 login_process_groups_create();
614 e60620644af3 login_process_per_connection = yes scales now better when multiple users are Timo Sirainen <tss@iki.fi> parents: 613 diff changeset	743
4148 f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	744 for (group = login_groups; group != NULL; group = group->next) {
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	745 if (login_group_start_missings(group) < 0) {
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	746 login_processes_stall();
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	747 return;
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	748 }
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	749 }
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	750
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	751 if (logins_stalled) {
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	752 /* processes were created successfully */
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	753 i_info("Created login processes successfully, unstalling");
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	754
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	755 logins_stalled = FALSE;
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	756 timeout_remove(&to);
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	757 to = timeout_add(1000, login_processes_start_missing, NULL);
f60d73c96618 If master process runs out of file descriptors, try to handle it more nicely Timo Sirainen <tss@iki.fi> parents: 3901 diff changeset	758 }
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	759 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	760
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	761 static int login_process_send_env(struct login_process *p)
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	762 {
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	763 extern char **environ;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	764 char **env;
3607 a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	765 ssize_t len;
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	766 int ret = 0;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	767
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	768 /* this will clear our environment. luckily we don't need it. */
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	769 login_process_init_env(p->group, p->pid);
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	770
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	771 for (env = environ; *env != NULL; env++) {
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	772 len = strlen(*env);
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	773
3607 a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	774 if (o_stream_send(p->output, *env, len) != len \|\|
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	775 o_stream_send(p->output, "\n", 1) != 1) {
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	776 ret = -1;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	777 break;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	778 }
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	779 }
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	780
3607 a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	781 if (!p->group->set->login_chroot) {
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	782 /* if we're not chrooting, we need to tell login process
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	783 where its base directory is */
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	784 const char *str = t_strdup_printf("LOGIN_DIR=%s\n",
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	785 p->group->set->login_dir);
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	786 len = strlen(str);
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	787 if (o_stream_send(p->output, str, len) != len)
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	788 ret = -1;
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	789 }
a2dd3d895e00 Several fixes to make running from inetd working again Timo Sirainen <tss@iki.fi> parents: 3384 diff changeset	790
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	791 if (ret == 0 && o_stream_send(p->output, "\n", 1) != 1)
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	792 ret = -1;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	793
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	794 env_clean();
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	795 return ret;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	796 }
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	797
3863 55df57c028d4 Added "bool" type and changed all ints that were used as booleans to bool. Timo Sirainen <tss@iki.fi> parents: 3692 diff changeset	798 static bool login_process_init_group(struct login_process *p)
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	799 {
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	800 p->group->processes++;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	801 p->group->listening_processes++;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	802
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	803 if (login_process_send_env(p) < 0) {
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	804 i_error("login: Couldn't send environment");
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	805 return FALSE;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	806 }
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	807
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	808 return TRUE;
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	809 }
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	810
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	811 static void inetd_login_accept(void *context __attr_unused__)
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	812 {
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	813 struct login_process *p;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	814 int fd;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	815
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	816 fd = net_accept(inetd_login_fd, NULL, NULL);
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	817 if (fd < 0) {
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	818 if (fd < -1)
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	819 i_fatal("accept(inetd_login_fd) failed: %m");
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	820 } else {
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	821 net_set_nonblock(fd, TRUE);
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	822 fd_close_on_exec(fd, TRUE);
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	823
1275 af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	824 p = login_process_new(NULL, ++login_pid_counter, fd);
af685269ead0 login: Wait until we're connected to auth process before executing command Timo Sirainen <tss@iki.fi> parents: 1273 diff changeset	825 p->initialized = TRUE;
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	826 p->inetd_child = TRUE;
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	827 }
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	828 }
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	829
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	830 void login_processes_init(void)
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	831 {
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	832 auth_id_counter = 0;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	833 login_pid_counter = 0;
1055 a72bba3f8a55 Rewrote setting handling. Changed some existing settings also since POP3 Timo Sirainen <tss@iki.fi> parents: 1039 diff changeset	834 login_groups = NULL;
613 1906116a62ce Finally support for handling each login connection in it's own process. Timo Sirainen <tss@iki.fi> parents: 596 diff changeset	835
945 501f076f2e74 Rewrote hash table code, works with less memory now. Also some memory Timo Sirainen <tss@iki.fi> parents: 929 diff changeset	836 processes = hash_create(default_pool, default_pool, 128, NULL, NULL);
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	837 if (!IS_INETD()) {
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	838 to = timeout_add(1000, login_processes_start_missing, NULL);
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	839 io_listen = NULL;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	840 } else {
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	841 to = NULL;
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	842 io_listen = io_add(inetd_login_fd, IO_READ,
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	843 inetd_login_accept, NULL);
2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	844 }
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	845 }
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	846
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	847 void login_processes_deinit(void)
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	848 {
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	849 if (to != NULL)
3879 928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	850 timeout_remove(&to);
1273 2cf2e08a6ee9 Somewhat working code to support loading Dovecot from inetd and such. It Timo Sirainen <tss@iki.fi> parents: 1237 diff changeset	851 if (io_listen != NULL)
3879 928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	852 io_remove(&io_listen);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	853
4538 9d9e72374164 Fixes to login process handling, especially with Timo Sirainen <tss@iki.fi> parents: 4388 diff changeset	854 login_processes_destroy_all(TRUE);
0 3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	855 hash_destroy(processes);
3b1985cbc908 Initial revision Timo Sirainen <tss@iki.fi> parents: diff changeset	856 }

Mercurial > dovecot > original-hg > dovecot-1.2

annotate src/master/login-process.c @ 4631:f5d53a684856 HEAD