Commit Briefs

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.


Omar Polo

don't have the config being a global


Omar Polo

move hosts into the config struct


Omar Polo

move fastcgi from global var to the config struct

while here also make them a list rather than a fixed-size array.


Omar Polo

remove foreground / verbose from config

set them as global vars; rename foreground -> debug


Omar Polo

remove forgotten include of logger.h


Omar Polo

move log_request to gmid.c

so that ge can provide its own log_request without requiring a separate logger process.



Omar Polo

rework the daemon to do fork+exec

It uses the 'common' proc.c from various OpenBSD-daemons. gmid grew organically bit by bit and it was also the first place where I tried to implement privsep. It wasn't done very well, in fact the parent process (that retains root privileges) just fork()s a generation of servers, all sharing *exactly* the same address space. No good! Now, we fork() and re-exec() ourselves, so that each process has a fresh address space. Some features (require client ca for example) are temporarly disabled, will be fixed in subsequent commits. The "ge" program is also temporarly disabled as it needs tweaks to do privsep too.


Omar Polo

absolutify the path to the pid file


Omar Polo

switch to the more usual log.c