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

More KNF.

Add SSL_OP_NO_TLSv1_1 and SSL_OP_NO_TLSv1_2
Document that SSL_OP_NO_SSLv2 is a no-op now

-option ALTQ
ALTQ has served us well for years and was extremely important not just for
us, but for the entire bandwidth management arena. Back when we got altq,
the subject was not yet well researched and understood, which is why altq
is the framework with pluggable schedulers it is. Kenjiro Cho (kjc@) did an
amazing job there.
Now, more than 10 years later, we do have a good understanding and can use
a simpler framework with just one priority queueing and one bandwidth
shaping mechanism each - the new queueing subsystem. Last not least because
it is incredibly painful to maintain both in parallel, it is time for altq
to depart. Farewell, thanks for many years of good service. Everybody
using any form of "not just fifo" queueing owes Kenjiro a lot. At least
buy him a beer when you meet him.
And, allow me this personal note, thanks Kenjiro, working with you on the
topic has always been a great pleasure and I learned a lot from you. Thanks!

use an appropriate name for this variable.

ok millert@

do not mark od(1) as deprecated.

ok jmc@, millert@

More KNF and style consistency tweaks

tiny fix: Remove duplicate rows, they appeared after importing less 444
suggested Daniel Dickman

More KNF.

More KNF.

More KNF.

missing wildcard; pointed out by naddy@

move scsi_xs_put after checks that use fields in the xs

ok dlg@

implement emc_mpath_checksense() according to what my cx500 throws.

tested by jmatthew@

Two minor tweaks regarding the fallback from -u/-d to default mode:
(1) Use all files found on the command line, but do *not* use all stray
files found during fallback tree recursion.
(2) If the fallback works, call that success, i.e. exit(0).
As pointed out by naddy@, the latter is required for ports' happiness.

Properly handle symlinks (hardlinks and .so only files were already ok):
Use the file name of the symlink but the inode number of the file pointed to,
such that we get multiple mlinks records but not multiple mpages records.
Also make sure they do not point outside the tree we are processing.

Issue found by kili@ in desktop-file-edit(1), thanks!

use intrinsic strlcpy and strlcat everywhere so we only have one set of
funcitons to check for incorrect use. keep BUF_strlcpy and BUF_strlcat
for API comptibility only.
ok tedu@

OpenSSH 6.5 and 6.6 have a bug that causes ~0.2% of connections
using the curve25519-sha256@libssh.org KEX exchange method to fail
when connecting with something that implements the spec properly.

Disable this KEX method when speaking to one of the affected
versions.

reported by Aris Adamantiadis; ok markus@

OPENSSL_gmtime() is really just gmtime_r(); ok guenther

sync

spacing

tweak;

There is no need to initialize globals to 0.

flesh out STANDARDS;

- merge two sections on error messages and interrupts
- move some non-standard sections into a more general CAVEATS
- some macro cleanup

remove references to rshd;

tone down some XXXXX to not appear in grep

rename wd33c93 to wd33c93ctrl (but keep the filenames as is) so we can
add attributes to it later; as wd33c93 is not a valid device name.

ok miod@

In update mode, when opening the database fails, probably because it is
missing or corrupt, just rebuild it from scratch.  This also helps when
installing the very first port on a freshly installed machine
and is similar to what espie@'s classical makewhatis(8) did.

Issue reported by naddy@ via kili@.

XXXXXXXXXXXXXXXX -> XXX
XXXXXXXXXXXXXXXXXXXXXXX -> XXXX

fix SEE ALSO;

unifdef NO_SOCK

round up some enemy sympathizers found calling RAND_seed().
ok beck reyk

now that knf carpet bombing is finished, switch to hand to hand combat.
still not sure what to make of mysteries like this:
for (i = 7; i >= 0; i--) {      /* increment */

For the WSDISPLAY_COMPAT_USL protocol, send the synchronizing signals to
the process, not just the thread.

ok kettenis@

Since we've been making heavy use of unifdef recently: update it to the
recent 2.10 release.

"This code was derived from software contributed to Berkeley by Dave Yost.
It was rewritten to support ANSI C by Tony Finch. The original version
of unifdef carried the 4-clause BSD copyright licence. None of its code
remains in this version (though some of the names remain) so it now
carries a more liberal licence."

ok deraadt@

Unsurprisingly, since <unistd.h> was so darn hard to find for OpenSSL developers
they had resorted to manually protyping read(2) instead of incredible amount of
preprocessor wizardry needed to find the ever illusive <unistd.h>. Let's just
include <unistd.h> and we don't need to do this.. While we're at it flense
out _OSD_POSIX and __DGJPP__ cruft.
ok krw@

ECDSA signature computation involves a random number. Remove the test trying to
force what RAND_bytes() will return and comparing it against known values -
I can't let you do this, Dave.

raise file limit to something more web scale, but lower connections so
there are some files to spare for other things.

unbreak tree - this was not the rand.c I was looking for

collateral damage

no app_rand.c

$HOME/.rnd will never be a good source of entropy. ok beck

Do not ask the user to pass either -DB_ENDIAN or -DL_ENDIAN to the compiler,
but rather figure out the endianness from <machine/endian.h> automagically;
help from guenther@

ok jca@ guenther@ beck@ and the rest of the `Buena SSL rampage club'

Shrink a local buffer to the size it really needs to be; this is the only
discrepancy found while checking proper {HEX,DECIMAL}_SIZE macro usage, which
is confusing enough.
tweaks and ok jca@, ok guenther@

remove bdes(1) so as to not encourage its use; if someone really
wants to use DES, there's another way.

ok deraadt sthen sobrado (and probably tedu)