Commit Briefs


Omar Polo

open the log files inside the chroot


Omar Polo

reopen log files upon SIGUSR2



Omar Polo

change log_request to take the code and meta unpacked

don't know what i was smoking when I wrote log_request() like that...




Omar Polo

implement `listen on'

Listening by default on all the addresses is so bad I don't know why I haven't changed this before. Anyway. Add a `listen on $hostname port $port' syntax to the config file and deprecate the old "port" and "ipv6" global setting. Still try to honour them when no "listen on" directive is used for backward compatibily, but this will go away in the next next version hopefully. At the moment the `listen on' in server context don't filter the host, i.e. one can still reach a host from a address not specified in the corresponding `liste on', this will be added later.



Omar Polo

typo; use the `l' variable not `len'


Omar Polo

add a privsep crypto engine

Incorporate the OpenSMTPD' privsep crypto engine. The idea behind it is to never load the certificate' private keys in a networked process, instead they are loaded in a separate process (the `crypto' one) which signs payloads on the behalf of the server processes. This way, we greatly reduce the risk of leaking the certificate' private key should the server process be compromised. This currently compiles only on LibreSSL (portable fix is in the way).


Omar Polo

parse the config file only once

Don't have all the processes read gmid.conf. The parent needs to do that, and the will send the config to the children (already happening.) The other processes were reading the config anyway to figure out the user and the chroot (if enabled); make the parent pass additional flag to propagate that info. We dissociate a bit from the "usual" proc.c but it's a change worth having.




Omar Polo

parse_conf: don't die on error, return -1

this avoids having the daemon dieing on SIGHUP with a bad config file.