more const -> sizeof()
no binary changes

ok gilles@

first steps towards better mta code. currently mta uses struct batch to
store a lot of its session related code, but this is just not right and
this commit starts making mta code aware of struct sess

first steps towards better mta code. currently mta uses struct batch to
store a lot of its session related code, but this is just not right and
this commit starts making mta code aware of struct session. This will
ease the implementation of ssl sessions in mta.

while at it, make mta autodetect port to use if it isn't provided in a
rule but can be derived from a parameter (i.e: "relay via ssmtp ...").

show more ...

slightly change "relay via" so that it can differentiate "ssmtp", "tls" and
"ssl" while providing mta with the informations it needs to do its work.

- change name of "masked" member in struct netaddr, it was misleading
- allow "from all" so that the ugly "accept from { 0.0.0.0/0, ::/0 }"
construct becomes a nice looking "accept from all"

ok jace

- change name of "masked" member in struct netaddr, it was misleading
- allow "from all" so that the ugly "accept from { 0.0.0.0/0, ::/0 }"
construct becomes a nice looking "accept from all"

ok jacekm@

show more ...

- import first bricks of SMTP AUTH support. currently only AUTH PLAIN is
supported, AUTH LOGIN will follow soon. AUTH will only work if a
listen directive has "enable auth" keywords, AND session is

- import first bricks of SMTP AUTH support. currently only AUTH PLAIN is
supported, AUTH LOGIN will follow soon. AUTH will only work if a
listen directive has "enable auth" keywords, AND session is safe
(ssmtp or starttls).

show more ...

- condition lists is wrongly described, unbreak the following syntax:
"accept for { domain "foo", domain "bar" } ..."
From Nicholas Mariott <nicholas.marriott@gmail.com>

- last snprintf -> bsnprintf

- snprintf -> bsnprintf

That the "aliases" and "virtual" maps satisfy m_src == S_DB is checked
too late, ie. at alias resolution time, and it's only a log_info.

Move the check to parse.y, and make daemon die if m_src != S_

That the "aliases" and "virtual" maps satisfy m_src == S_DB is checked
too late, ie. at alias resolution time, and it's only a log_info.

Move the check to parse.y, and make daemon die if m_src != S_DB.

ok gilles@

show more ...

Unbreak -Werror.

ok gilles

- it is now possible to specify an interface instead of an address or a
hostname in a listen statement (ie: listen on lo0)
request by deraadt@ a while ago, ok jacekm@

Declare printf-style functions with __attribute__((format(printf,x,x)))
and fix some of the errors caught by this. Part of a general push to
make yyerror() -Wformat clean throughout the tree.

obvious 'missing space' typo in message, ok gilles@ krw@

- recent change in parse.y caused htons() to be called twice on the port
provided to "relay via" rules, once in parse.y once in lka.c, fix.
- rename struct address to struct relayhost, introduce str

- recent change in parse.y caused htons() to be called twice on the port
provided to "relay via" rules, once in parse.y once in lka.c, fix.
- rename struct address to struct relayhost, introduce struct mxhost which
not only holds the sockaddr_storage, but also additionnal flags we
want forwarded to the mta process.
- propagate the change

show more ...

- allow the optionnal ssmtp keywork in "relay via" rules, while at it
allow port to become optionnal (implicit 25) or provided by value
or name.

- rephrase the "cannot load cert" warning that is output at startup when a
listen directive has no matching certificate. it sounds like a
critical failure when it just means "no tls support".
- min

- rephrase the "cannot load cert" warning that is output at startup when a
listen directive has no matching certificate. it sounds like a
critical failure when it just means "no tls support".
- minor log_debug() addition in smtp.c

show more ...

- define MAX_LINE_SIZE which is the maximum length of a line we allow from
a client. it must be set to the highest value we have from all of
the extensions which are/will be implemented.
- replace

- define MAX_LINE_SIZE which is the maximum length of a line we allow from
a client. it must be set to the highest value we have from all of
the extensions which are/will be implemented.
- replace all occurences of STRLEN define with MAX_LINE_SIZE, kill STRLEN

show more ...

spaces fixed while reading code

- move '=>' into the lex loop, requested by and with help from deraadt@

- snprintf() can return -1, make sure every call is checked properly

- recognize '=>' as one token instead of trying to match '=' and '>'. this
prevents: "foo = > bar" from being valid

add a few missing id tags; there are a bunch of files, and developers
will probably miss this change when working on more important matters,
so it is probably better to sort them now. there is a ris

add a few missing id tags; there are a bunch of files, and developers
will probably miss this change when working on more important matters,
so it is probably better to sort them now. there is a risk of losing
the tags if a change needs to be reverted too.

written with excellent advice from jmc@

ok gilles@

show more ...

- put back all copyright holders
- add myself as a copyright holder

smtpd is a smtp server implementation for OpenBSD. It is a work in progress
which still lacks many features. bringing it in tree will help working on it
more easily.

"at this stage it should go in"

smtpd is a smtp server implementation for OpenBSD. It is a work in progress
which still lacks many features. bringing it in tree will help working on it
more easily.

"at this stage it should go in" henning@, "move ahead" deraadt@

show more ...