CVSROOT: /cvs
Module name: src
Changes by: e...@cvs.openbsd.org 2012/01/13 07:01:58
Modified files:
usr.sbin/smtpd : mda.c mta.c queue.c runner.c smtpd.c smtpd.h
Log message:
Stop using envelope->status to report delivery outcome to the
runner/queue. Instead, replace IMSG_QUEUE_MESSAGE_UPDATE with three
messages:
- IMSG_QUEUE_DELIVERY_OK
- IMSG_QUEUE_DELIVERY_TEMPFAIL
- IMSG_QUEUE_DELIVERY_PERMFAIL
1) it's less confusing as status is also used by smtp
2) it's easier to see what happens just looking at imsg traces
3) it makes the code path generally easier to follow
4) it's safer because it enforces clear semantics and intent, whereas
the status field is loosely defined and could carry bogus values.
ok gilles@ chl@
