Age | Commit message (Collapse) | Author |
|
same spelling fix from jmc and gilles
ok gilles
|
|
OK gilles@
|
|
table has to be defined BEFORE
consequently move all tables in the examples to the beginning and before the
listen directive to avoid tables not being found
no functional change
ran into this myself earlier, also reported by cjones via irc
ok gilles
|
|
runtime state is not serialized with the envelope, so add it to the imsg.
ok gilles@
|
|
default since it is computationally expensive and a potential DoS vector.
ok gilles@
|
|
|
|
original diff from Bernard Spil
ok millert
|
|
from Boudewijn Dijkstra
|
|
diff from david+bsd
i also removed Op, since ... is enough/
|
|
correctly - logically complete that now by removing MLINKS from base;
authors need only to ensure there is an entry in NAME for any function/
util being added. MLINKS will still work, and remain for perl to ease
upgrades;
ok nicm (curses) bcook (ssl)
ok schwarze, who provided a lot of feedback and assistance
ok tb natano jung
|
|
the identical io_set_blocking() and io_set_linger().
Since both are always called to turn off blocking or lingering,
nuke the parameter and associated enum in favour of "just doing the
right thing".
While passing remove the unneeded last parameter to the remaining
fcntl(F_GETFL).
Finally, rename the functions to io_set_nonblocking() and
io_set_nolinger() for clarity.
No functional change.
Started with a sweep of fcntl() usage inspired by guenther@.
ok gilles@
|
|
Just NULL is not good practise as NULL is theoretically allowed to
be an integer rather than a pointer.
Use (char *)NULL consistently instead of scattering a few (char *)0
and (void *)NULL into the mix.
Prompted by and probably ok deraadt@ millert@ kettenis@
Definitely ok mestre@ ratchov@
|
|
From: Sevan Janiyan <venture37@geeklan.co.uk>
|
|
override the value provided by the user if any. this commit moves the
initialization earlier to avoid this.
issue reported by and fix ok jturner@
|
|
|
|
|
|
There's no need to have a separate case for non-CIDR addresses using
inet_pton(3) as inet_net_pton(3) handles them as well.
This simplification is valid for all our daemons parsing addresses,
it is a common patern.
ok gilles@, millert@, jung@
|
|
|
|
original diff from Peter Bisroev <peter@int19h.net>
rewording from me, linebreaks from jung@
|
|
"listen on socket". this simplifies a bit of code, removes some special
cases and will allow attaching filters & masking source just as on lo0.
diff from Peter Bisroev <peter@int19h.net>
ok gilles@, jung@
|
|
diff from Peter Bisroev <peter@int19h.net>
|
|
work because it requires dbopen() to be the call that creates the db, while
we want mkstemp() to create an empty db ourselves.
it has never worked, no one could ever create a recno db with our makemap &
no one actually noticed so drop it.
ok millert@, sunil@, jung@
|
|
Avoid multiple getopt(3) evaluations without optreset set and
unspecified behavior of optind = 0 as per POSIX. Instead, iterate
over argv the first time for classification. Fixes -portable without
any divergence from -current, and all newaliases opts are handled
in one place.
Looks good to me millert@
Ok gilles@ jung@ eric@
|
|
a switch statement for improved readability. OK jung@ sunil@ gilles@
|
|
move to destination when done. this causes dbopen() to be called with an
empty file which happens to... not work with other db libraries.
adding O_TRUNC is noop for us, fixes the issue for other libraries and
lets us reduce delta between us and portable.
ok sunil@, ok eric@
|
|
|
|
Fix and simplify imsg_read() return value checking into a single
if..else block.
Ok gilles@ jung@
|
|
diff from Freddy DISSAUX, ok gilles@, jung@, sunil@, millert@
|
|
suggested by gilles
|
|
ok gilles
|
|
ok gilles@ jung@
|
|
ok gilles@ sunil@ jung@
|
|
if any (it happens when a message transaction is cancelled).
ok gilles@ sunil@
|
|
ok gilles@ sunil@
|
|
ok gilles@ jung@
|
|
smtp session. Simplify code while here.
ok gilles@ jung@
|
|
they can't end up uninitialized after the mkstemp/fdopen dance
|
|
we no longer prepend status code to "errorline". Fixes mismatch
between DSN's subject line and its content.
Ok jung@ gilles@ millert@
|
|
they are unused in OpenSMTPD and lead to crashes in -portable when we're
linked to OpenSSL starting with 1.0.2f
ok reyk@
|
|
titles (including flags) to distinguish between daemons, this makes it
possible to manage multiple copies of a daemon using the normal infrastructure
by symlinking rc.d scripts to a new name. ok jung@ ajacoutot@, smtpd ok gilles@
|
|
Ok eric@ millert@
|
|
|
|
filter_drain_query() can be called when the ->current filter is
still running the query. Just make sure we move forward after
having received the response from the current filter.
ok gilles@
|
|
not supported anyway.
ok sunil@ gilles@
|
|
ok gilles@ jung@
|
|
Ok jung@ gilles@ eric@
|
|
query-chaining machinery.
ok jung@ gilles@
|
|
are forked at startup
i thought i had committed this already, diff has been okayed several weeks
ago...
ok jung@, ok millert@, ok sunil@
|
|
the configuration file of a table, user actually understands what happens
diff by Alexis Vachette, ok jung@
|
|
all from tj
ok millert
|