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)

Remove the dead KAME code that dealt with IPv4-mapped IPv6 addresses.
Add a check for IPv4-mapped IPv6 destination addresses, like in the most
recent KAME code, for non-connected sockets.  This prevents packets from
reaching the wire through the default route, if a reject route
for ::ffff:0.0.0.0/96 isn't present.  ok claudio@

This remnant comment doesn't belong here.  ok claudio@

eroMgib dne- nai 68xtnetelca  .s

Not welcome

typo

It seems a generation of programmers is aping OpenSSL. We need re-education
camps.  RAND_ is considered hamful, we should not *re-implement* it here.
"fire bomb it" - tedu@, "dresdenizing" - beck@, "SSLaughterhouse five" miod@

Use the cleaned up asprintf-based make_config_name() to make the name of
the config file instead of the malloc/BUF_strlcpy/BUF_strlcat calls with
no return value checks (that make_config_name() also used to do prior to
being cleaned up).

ok beck@

first round of static config. ok miod

Check the return value of make_config_name() before attempting to use
the config filename.

ok beck@

another

another "string to make the random number generator think it has entropy"

delete "string to make the random number generator think it has entropy"

Put back i2d_ASN1_SET() and d2i_ASN1_SET() from the NO_ASN1_OLD prune, as there
are still some 3rd-party code using it, and fixing them is not trivial.

As an excuse gift, the memory leaks on failure in resurrected a_set.c have
been fixed.

RAND_xxx considered harmful. use arc4random_buf instead of nasty stuff.
ok tedu@

Wrap long lines.
OK lteo@, benno@

Move ident / perturb initialisation up, this is AF independent.
OK benno@

sync to traceroute6: use getnameinfo for destination ip
OK benno@

Declare socklen_t len in main, it's used in two places, no need
to declare it twice. We can get rid of a { } block.
OK benno@

Replace fprintf(stderr, ..); exit() with errx() and fprintf(stderr, ...)
with warnx()
OK lteo@, benno@

Drop rh0 support (-g), it doesn't work anyway.
OK benno@

Use getaddrinfo to resolve destination. I kept the inet_aton so the
great old ones can still traceroute 010.010.010.010.
OK benno@

RAND_egd is considered harmful.  Unbreak the tree by making kerberos not use
it. The rest of the RAND_ horror in here needs checking.
ok deraadt@

Introduce some regress tests against our routing table. At least that way
there is a chance that we do not break the network stack even more.
These regress tests already found a few issues.
The framework is ugly and does not properly recover from failures. Somebody
more skilled can come up with a better solution.
mpi@, blambert@ and sthen@ support this

this file is not relevant

We do have SO_SNDBUF.
OK benno@

We do have SO_SNDBUF and IP_HDRINCL.
OK benno@

replace perror(3) with err(3)/warn(3)
OK lteo@, benno@

Structure wait_for_reply() loop like traceroute, thereby moving the
loop body one indent layer up.
OK benno@

move cast from packet to ip up to avoid casts in print()
OK benno@

move ICMP6 code parsing to function
OK benno@

move ICMP code parsing to function
OK benno@

sync to traceroute:
* s/Dst/to/
* s/Src/from/
* drop Rcv
OK benno@

If -s is not given do a dummy connect to get outgoing ip,
unconditionally try to bind to this ip and get a source port for udp
this way, like traceroute6 is doing.  This means you can no longer
traceroute from IPs not present on the system. (There are probably
better tools if you want to send traffic from spoofed IPs.)
OK benno@

fix previous

Sync to tracroute: don't print source IP if -s is not given
OK benno@ (who wants it back in some form after the merge)

guenther would prefer more separation

spacing

Sync to tracroute: handle "time exceeded in transit" before the
switch and add a default case.
OK benno@

sync packet_ok signature to traceroute6
OK benno@

sync to traceroute: s/opacket/packetdata/
OK lteo@, benno@

Embed struct tv32 into struct opacket like traceroute.
This changes the data part of an icmp6 paket, before it only
contained the timestamp, now it contains a whole struct opacket.
Shouldn't be an issue as nobody looks at this data anyway.
OK benno@

Malak: I think we made the merchant angry.
Conan: Are you surprised?
Malak: But we didn't steal everything he had!
Conan: We didn't have time.

Factor out build_probe{4,6} from send_probe; now send_probe is
AF independent. While there define outpacket as u_char and
cast as needed in traceroute6.
OK benno@

sync

millert said i can kill rshd

we need to crank

Fix SSL client-only mode when no RSA private key is needed.

Found by andre@ with the args-ssl-server.pl regress test.

ok andre@

remove include files not needed

Document support for "openssl s_client -starttls lmtp"

Finish zapping SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION usage; only keep
the #define for compat, but document that it's a no-op now.  Also, neuter
the -legacy_renegotiation option to "openssl s_{client,server}"

ok beck@

use the portable construct around asprintf; pointed out by halex

reaching into altq outside #ifdef ALTQ is bad, mmkay? ok claudio

tcp_respond: let the stack worry about the cksum instead of doing it
manually, ok naddy (in january)

pf_send_tcp: ask the stack to do the cksum instead of doing it manually
ok benno lteo naddy (back in january)

It's been a quarter century: we can assume volatile is present with that name.

Some dude named Tavis Ormandy reported a bug which has gone unfixed.
http://marc.info/?l=openssl-users&m=138014120223264&w=2
Arguably a doc bug, but we argue not. If you parse a new cert into memory
occupied by a previously verified cert, the new cert will inherit that
state, bypassing future verification checks. To avoid this, we will always
start fresh with a new object.

grudging ok from guenther, after i threatened to make him read the code yet
again. "that ok was way more painful and tiring then it should have been"

reaching into altq unconditionally (and w/o ifdef ALTQ) is bad, mmkay?

since e_os.h is dead, and e_os2.h is installed, we can fetch from there.
This means we don't need the reach-around anymore.

It's been a quarter century: we can assume volatile is present with that name.

Put the final pieces from e_os.h in the required places, and remove it.
"dance on it's grave" says beck
ok guenther beck

cut altq here