Disengaging imsgev is tricky. First imsgev_clear as we know there isn't anything left to be written. Signal imsgev termination by setting iev->terminate = 1(Mind that we are setting it from with the callback). As imsgev would again callback dispatch with IMSGEV_DONE, we cannot free imsgev yet. So iev_maildrop need to exist beyond lifetime of session until IMSGEV_DONE. Allocating it seperately from session. imsgev_close is avoided as it schedules another EV_WRITE which is not needed in our case. This fixes a crash observed consistently with a little perl script to simulate concurrent sessions. Thanks MALLOC_OPTIONS='SFG<<'

Shorten.

space nit.

No use of iev->data, set it to NULL.

Give session a chance to init its imsgev. Let session initiate the process of MAILDROP_INIT. Maildrop now sets up itself and waits for MAILDROP_INIT from session. This would now eliminate the doubt in the earlier setup when priv process forks and the child gets cpu slice first before parent which sends MAILDROP_INIT to an uninitialized imsgev.

mtype, mpath are accessible to maildrop, simplify setup parameters.

Rename to differentiate setup and init operations.

Bounds check array index. Fixes a crash when len == 1.

First clear io then iobuf.

Latest changes from upstream with commit message... When the three possible return values are -1, 0, and 1, != 1 is the same as <= 0. And the latter is the normal idiom so use that. ok claudio@ henning@

Prepend session id to the log message.

knf and whitespace fixes.

Fix a crash during concurrent sessions TIMEOUT reported by James Turner (#5). imsgev_close schedules(event_add) iev and the event struct is free-ed before the event is dispatched. Include string.h after recent libressl changes.

Pull in latest changes for iobuf/ioev from smtpd.

This cannot truncate unless new states are added.

Equal sized buffer copy, no truncation happens.

args are checked for size in parse function. Cast to void as no truncation check required here.

hash is a fixed length string. Copying is always done to same length buffer. Explicitly cast to void for clarity.

Update README.md

Mark uidl correctly in list_res for the terminal. Fixes UIDL command on an empty maildrop.

Remove redundant sock member in session. io already holds the sock for each session.

pull in latest master.

Kill pop3_main return value as we now waitpid on WAIT_ANY.

Since we now waitpid on WAIT_ANY, pop3_main needed return pid.

Merge pull request #3 from jturner/killzombies Make sure zombie processes are not left over

Simplify maildrop path handling. Regardless of the order of options path is now correctly handled.

Make sure zombie processes are left over

Fix clang warnings. offset is always long.

Return correct nlines for RETR in maildir handling.

Use correct format of listing.

Move it closer to list.

Merge branch 'refactor'

merge from cvs.

Import.

Properly NUL terminate expanded path by memsetting it.

Simplify list_all handling. Send an imsg for each msg instead of consolidating into one imsg. We are no longer limited by MAX_IMSGSIZE. End of list is signalled by sending an imsg with hash as "" or sz as 0.

Create README.md

Import pop3d.