knf

add a missing strlcpy() check in MAIL FROM's DSN parameters parsing, the
truncation would lead to a failure later in the code path but we can fail
earlier with a nice enhanced status code

More KNF.

(void) cast strlcpy() calls that cannot truncate (copies between buffers of
same size with a truncation check on the initial buffer)

certs are looked up by hostname, the size of the buffer should use the
max hostname len, not max pathname len as before

(void) cast a strlcpy() that cannot truncate because lka.c ensures the
reply will fit in the buffer before returning the answer

Simplify things by using the return code of ask_yn() directly instead
of looking in resp. No need to specify 'no' as default answer. It's
the default for ask_yn() anyway.

OK krw@ halex@

More KNF.

tweak previous;

(void) cast strlcpy() that cannot truncate, dns.c ensures the reply will
fit in the buffer before returning the answer

stop "advertising" disabling pmtud and window size increasing
very rarely if ever needed any more. we should not trick people into
thinking they are impoving sth doing so, it's rather the opposite
these days.
ok claudio

need to escape %U, since it is used in Re/Rs blocks;

sync

zap trailing whitespace;

remove the openssl_fdset wrapper, and a variety of VMS'ism's like
crazy (void *) casts all over the place
ok beck jsing

(void) cast strlcpy() calls that cannot truncate or where the source data
has a length check right before the copy

KNF.

Remove a gethostbyname() cache layer.  The internet works better these
days.  Initially fixed this, but Ted asked for it to die.

ReadFile() and GetStdHandle() are not very POSIX.
ok beck jsing

(void) cast a snprintf() call that cannot truncate

Add missing description for IPv6 mtudisctimeout sysctl and
rework the wording for both IPv4 and IPv6.

OK from sthen@, henning@ and claudio@

Dead stores in subr_hibernate.c

truncation check some snprintf calls (over-cautiously in case)
ok jsing beck

can't actually do this. cause unknown.

bye bye

use "!received-on any" to absolutely ensure that we're not forwarding
carp, rpc or nfs traffic in the initial ruleset active during network
startup for a short time (or a much longer time if /etc/pf.conf is
screwed up). ok phessler

disgusting altq compat hack goes away

altq includes are gone
yes, we normally leave this for theo, but this triggered my final
"any altq bits left" check and I had to wait for a build anyway...

stop installing altq includes

now that if_snd is a proper ifqueue, this cast dies

ifnet's if_snd becomes a regular ifqueue instead of ifaltq, the need to
keep ifqueue and ifaltq in sync is gone and thus the comment obsolete,
and finally there is no more need to include if_altq.h either

minus altq tentacles, plus missing IF_DROP here as well

Try to be clear why struct mrt_config nodes are allocated and
assigned to struct mrt pointers. I may need to rethink this later
but lets hope this does not confuse more people and analyzers.

Fix some serious pointer-arithmatic-magic-number-unchecked-return eyebleed
that I stumbled into here and got stuck with.  If modern society can get past
selling daughters for cows, surely we can decide to write modern C code in
an "application" that is probably 3 lines of shell/python/cgi away from
talking to the internet in a lot of places.. (This file still needs a lot
more love though)
"oh god yuck" deraadt@
ok tedu@

KNF.

sync

Repair some indent to be more KNF, and break instead of fallthrough to
default which does break

one small tweak to avoid ever going off the end of a string.

improved checking for invalid hashes. from solar designer

Once great men lived here... giants... gods... once, but long ago.

the valkyries have come for rsh as well

Delete futile calls to RAND_seed. ok djm

sigh, another driver fiddling with altq outside #ifdef ALTQ

kill altq bits here as well

(void) cast strlcpy() call that uses truncation as part of its parsing
logic to extract at most sizeof buf - 1 bytes from source

More KNF.

(void) cast a strlcpy that cannot truncate

(void) cast strlcpy/strlcat/snprintf calls that cannot truncate

remove altq bits here, too
(i was convinced i committed that yesterday already, hrm)

(void) cast strlcpy() calls that cannot truncate

(void) cast strlcat() and snprintf() that cannot truncate

be a bit more strict with an strlcat() truncation by causing it to fail in
table_create() instead of later in parse.y - in both cases, this would
cause smtpd to fatal() at startup if a table has a config file too large

(void) cast strlcpy() calls that cannot truncate

(void) cast a snprintf() call that can't possibly truncate unless we
suddenly supported a cipher with a name that's > 200 bytes long ...
... in which case harmless truncation would occur and we'd have to
readjust our buffer

Fix display of destination IP when host is an IP address.
Pointed out by and OK benno@

(void) cast snprintf calls that cannot truncate

(void) cast a strlcpy() that cannot truncate
(void) cast a few snprintf()/strlcat() that cannot truncate and that would
be harmless if they could

(cast) void two strlcpy() and a snprintf() that cannot truncate

(void) cast strlcpy/strlcat/snprintf calls that cannot truncate (and would
       be harmless in this case if they could)

kill a stray \

(void) cast strlcpy/strlcat/snprintf that cannot truncate

(void) cast strlcpy() and snprintf() that cannot truncate

(void) cast snprintf() and strlcat() calls that cannot truncate

(void) cast strlcpy() / strlcat() / snprintf() that cannot truncate

(void) cast a few calls that are ok to truncate because they are not used
in anything critical / unsafe, but just to make some debug/info logs a bit
more informative

Lacking a proof that--for this implementation--exposure of Montgomery
multiplication or RSA blinding parameters doesn't permit retroactive
timing analysis of the secrets, we'll do the stupidly cheap thing and
cleanse them before freeing them.

ok deraadt@

More KNF.

(void) cast some strlcat() calls that cannot truncate
add a few fatalx() calls at places where it shouldn't fail, we'll assess
which one may be relaxed later as this code is not finished nor plugged
yet.

Remove hacky workaround for Cray T3E.
ok guenther

More KNF.

oops, typo got into change

add missing checks to strlcpy() when copying envelope "destination" buffer
to the mda delivery buffer. we should never hit these unless we mistakenly
change the value of a define, better be safe than sorry.

(void) cast strlcpy/strlcat that cannot truncate or that we know and want
to truncate rather than lose (informative data not used by smtpd but
intended to help the human reading the log)

shrink pf by 445 lines.
while there, get rid of the altq ioctls and assciated now obsolete code

when receiving mail for user+tag@ and maildir is setup, we create a .tag
subdirectory in the maildir, add missing check to strlcat() so that if path
with .tag exceeds SMTPD_MAXPATHLEN we fail instead of creating a .tag dir
that's truncated.

(void) cast strlcpy()/strlcat() that cannot truncate

More KNF.

(void) cast strlcat(), they can't truncate

unchecked strlcat() call in expand_to_text() may cause aliases expansion to
fail later, make it fail early.

(void) cast other calls, they are detected and handled correctly later

Add missing man page descriptions for the following IPv6 sysctls:

-maxdynroutes
-maxifprefixes
-maxifdefrouters
-neighborgcthresh

OK from sthen@, claudio@ and henning@

malloc + strlcpy -> strdup

(void) cast strlcpy(), it cannot truncate
(void) cast strlcat(), they are detected and handled later

remove the altq enable/attach and disable/detach bits

bye bye altq bits

(void) cast this strlcpy(), it cannot truncate

another questionable "optimization": de used tulip_ifstart_one instead
of tulip_ifstart if the sendqueue was empty, but only if altq wasn't
compiled in (i. e., that's a _compile time_ decision and not based on
wether altq was actually used).
just use tulip_ifstart all the time, as before in our regular kernels.
kill tulip_ifstart_one completely.
makes sense to sthen

unifdef ENOTDIR, everyone has it

remove very questionable altq "optimization" - claudio and I can't
convince ourselves that that was right to begin with. anyway, begone.

all I wanted to do is removing the altq special casing, but then it turned
out the entire codepath is unreachable. glad I'm not our ppp maintainer, he
has work to do.
kill that unreachable code, with & ok claudio

remove altq special casing.
big WTF regarding the fastq use there while verifying w/ claudio, but
that's for the ppp maintainer and unrelated

Don't panic if we're trying to disestablish an MSI but the hardware is gone.

tested by & ok mlarkin@

We'll interpret a (void) cast on snprintf() to mean it's been verified that
truncation is either desirable, not an issue, or is detected and handled later

ok deraadt@

these strlcpy can't truncate, the copy is from a buffer to a buffer of same
size and the first buffer handles the truncation already

bye bye

cast (void) this strlcpy(), it can't fail copying a small constant string
into a much larger buffer

Use somewhat harsher language and better examples; demonstrate that
non-dangerous use functions is difficult.
ok guenther

when copying socket path, check that we didnt truncate it which would cause
the following connect() to fail.

the altq versions of the IFQ_* macros can finally go. chances of this
file becoming readable increase.

Demonstrate correct usage of snprintf (regarding overflow detection)
ok guenther

Split inet(3) into three pages by decade: 1980s -> inet_lnaof(3),
1990s -> inet_addr(3), 2000s and beyond -> inet_ntop(3).

ok tedu@ (who also noted the timeline) deraadt@ jmc@

it's ok for strlcpy to fail here though it can't, cast void

egd is gone

   /*
        * altq for loop is just for debugging.
        * only used when called for loop interface (not for
        * a simplex interface).
        */
bye bye!

The internal ssl2_* functions and variables are gone