calloc is better. from Peter Malone

calloc is better. from Peter Malone

repair missing semicolon. from Ian Mcwilliam

misplaced punctuation

Unifdef -UPKCS1_CHECK and remove SSL_OP_PKCS1_CHECK_[12], this is leftover
``debug'' code from a 15+ years old bugfix and the SSL_OP_PKCS1_CHECK_*
constants have had a value of zero since ages. No production code should use
them.

ok beck@

ain't nobody got time for hpux

fix some variables

replace a bunch of hand duped strings with strdup

Don't bother generating win64 assembly prologue.

malloc/memset -> calloc. from peter malone

No longer needed during build. Brings a smile on beck@'s face.

Audit malloc(3)/calloc(3)/realloc(3) usage.
* Change eight reallocs to reallocarray to be safe from overflows.
* Change one malloc to reallocarray to be safe from overflows.
* Change one calloc to reallocarray, no zeroing needed.
* Change the order of arguments of three callocs (aesthetical).

null pointers after free to prevent double frees and worse. also fix a
very obvious use after free. this file may still be a total loss.

The usual idiom to cope with systems not defining socklen_t is to add a
#define socklen_t int
somewhere (or a typedef, whatever gives you an integer type of the size
your system expects as the 3rd argument of accept(2), really).

OpenSSL here is a bit more creative by using an union of an int and a size_t,
and extra code if sizeof(int) != sizeof(size_t) in order to recover the
proper size. With a comment mentioning that this has no chance to work on
a platform with a stack growing up and accept() returning an int, fortunately
this seems to work on HP-UX.

Switch to the light side of the force and declare and use socklen_t variables,
period. If your system does not define socklen_t, consider bringing it back
to your vendor for a refund.

ok matthew@ tedu@

whitespace

close socket in failure cases too

if realloc failed, BIO_accept would leak memory and return NULL, causing
caller to crash. Fix leak and return an error instead. from Chad Loder

One last Dec C tentacle on alpha.

When I grow up, I want to write workaround for long long multiplications
under __TANDEM systems and compilers, using hardcoded octal numbers. NOT.

Remove IRIX_CC_BUG workaround.

I'm glad to know that Ultrix CC has a bug optimizing switch() statements
lacking an explicit `case 0:' construct. But Ultrix has been dead for more than
15 years, really. Don't give it any reason to move out of its coffin.

Grr, the more you remove, the more there remain.

Forgotten during latest spring cleaning of unreferenced bits.

Unifdef -UPEDANTIC. ok beck@ tedu@

Audit strlcpy(3)/strlcat(3) usage:
* Add missing truncation checks to three calls.
* In four cases where we know that the distination buffer is large enough,
cast the return vailue to (void).

Remove duplicate pointer assignment in ASN1_primitive_free(); Dirk Engling

Figure out endianness at compile-time, using _BYTE_ORDER from
<machine/endian.h>, rather than writing 1 to a 32-bit variable and checking
whether the first byte is nonzero.

tweaks and ok matthew@; ok beck@ tedu@

Remove more Kerberos cruft.

ok jca@ jmc@

Better name for fdmaskp, reduce the scope of some variables, don't abuse
cc for poll()'s return value, and indent decls.  ok florian@

improve SQL style: avoid "SELECT *", be explicit in what columns we want;
suggested by espie@.

tweak previous; sorry ingo, my review was pretty poor;

Audit strlcpy(3)/strlcat(3) usage.

* Repair three instances of silent truncation, use asprintf(3).
* Change two instances of strlen(3)+malloc(3)+strlcpy(3)+strlcat(3)+...
to use asprintf(3) instead to make them less error prone.
* Cast the return value of four instances where the destination
buffer is known to be large enough to (void).
* Completely remove three useless instances of strlcpy(3)/strlcat(3).
* Mark two places in -Thtml with XXX that can cause information loss
and crashes but are not easy to fix, requiring design changes of
some internal interfaces.
* The file mandocdb.c remains to be audited.

Fix format string in DEBUG message (not compiled by default).

ok blambert@

remove Z option and default to something halfway to J.
we always junk small chunks now, and the first part of pages,
but only after free. J still does the old thing. j disables everything.
Consider experimental as we evaluate performance in the real world.
ok otto

No need for vargs here.
While there move declaration of divert{,6}_output() to .c as it's a
private function. Also switch first two args to make it more like
similar functions (both suggested by mpi@).
Input/OK mpi@, OK lteo@

Use arc4random as PRNG backend, instead of libcrypto RAND.
Feedback and ok guenther@ deraadt@

Discussed with upstream, who is preparing an arc4random backend.

more info about HISTORY; ok jmc@

His name is Van Jacobson.  ok jmc@

Document RTS5227 support in rtsx(4).

don't record duplicate IdentityFiles

Casting from a const unsigned char ** to a const unsigned char ** seems...
unnecessary. Remove the temporary ugly casts - the comments even call them
that!

ok guenther@

We do not need ARGV, Argc and Argv - just use argc and argv like normal
people do.

ok beck@ (although he wanted #Arrrrrrrgv instead).

Replace malloc / memset / select song and dance with poll(2) like
ping6.
Prompted by and initial diff from Peter Malone (peter _AT_
petermalone _DOT_ org), thanks!
OK jca@

Don't use varargs for udp_output() and sync the argument order with
udp6_output().

ok henning@, reyk@, jca@

don't clear label before readasciilabel(), it will not fill in some
required fields like disk size, causing consistency check failures.
with and ok krw@

Kill useless assignement, matchlen is initialized two lines below.
From Fritjof Bornebusch.

Remove badly outdated information: FDDI and Token Ring are gone,
VLAN tags on transmitted packets have been visible to bpf since
the introduction of bpf_mtap_ether().  ok brad@, henning@

Don't attempt to deal with link types supported by no drivers in the
tree.  ok henning@

explain a bit more what's going on for stupid me.
okay otto@

Differentiate between linked and unlinked window closes and renames,
like we already do for adds. From Andre Masella.

Remove krb5 bits from rc(8).

ok reyk@

You don't want to use ifa_ifwithroute(), it exists for to the routing
crazyness only.

Merge traceroute6 into traceroute.
Not yet enabled in the build.
OK benno@

Prepare for merge: s/hops/ttl/ in struct packetdata
OK benno@

Prepare for merge: no more useicmp
OK benno@

KNF
OK benno@

bzero -> memset
OK benno@

s/lastaddr/lastaddr6/
OK benno@

Prepare merge: check AF
OK benno@

missing break; not reachable
OK benno@

No need to set protocol in hints.
OK benno@

zap redundant cast
OK benno@

Introduce check_tos to unclutter the main loop.
OK benno@

move AF independet setsockopts down
OK benno@

s/Nxt/nxt/
OK benno@

Add error checking to sysctl. While there pass in an int otherwise
it failes with ENOMEM in traceroute.
OK benno@

sync to traceroute: move sndsock creation up.
OK benno@

Prepare for merge: introduce icmp_code for the AF switch.
OK benno@

Prepare for merge: s/icmp_code/icmp4_code/ and use icmp_code for
the AF switch.
OK benno@

Prepare for merge: s/packet_ok/packet_ok6/ and use packet_ok
for the AF switch.

Prepare for merge: s/packet_ok/packet_ok4/ and use packet_ok
for the AF switch.

Prepare merge: s/to/to6/; s/from/from6/
OK benno@

Prepare merge: introduce struct sockaddr *from, *to to be used in
AF independet places.
OK benno@

Prepare merge: s/to/to4/; s/from/from4/
OK benno@

Make this compile with -Wall et al.
OK benno@

Better, cleaner hash function that computes the same on be and le archs.
Should improve sparc64 and other be archs. ok matthew@ miod@

Make libssl and libcrypto compile with -Werror
ok miod@

Make sure ret->name is NULL'ed before return when freeing.
from Dirk Engling <erdgeist@erdgeist.org>

Rather than sprinkling magical numbers everywhere, we can use sizeof()
for the size of a fixed size array.
From Dirk Engling <erdgeist@erdgeist.org>

rename some hardware revisions to match FreeBSD

add support for RTL8168EP
From Edward O'Callaghan via FreeBSD

While we always mask the revision with 0x7c800000 linux uses a table
that masks with either 0x7c800000 or 0x7cf00000 depending on the chip.

Some of the hardware revisions I previously added can't be matched with
the current mask, these are already handled by other revision defines
so remove them.

When masking the revision defines with 0x7c800000:

RL_HWREV_8106E_SPIN1 is the same as RL_HWREV_8106E
RL_HWREV_8168G_SPIN1 is the same as RL_HWREV_8168G
RL_HWREV_8168G_SPIN2 is the same as RL_HWREV_8168GU

Add support for RTL8168GU
from Rafael Neves

Unify the model name to LUNA-88K{,2}, that is considered the
`official' name.

ok jmc@ miod@

unifdef -UAES_LONG for we do not intend to run on platforms where int is smaller
than 32 bits.

Use calloc() instead of malloc(n * s) followed by memset().  Not
actually used on OpenBSD but changed to avoid false positives in
audits.  From Jean-Philippe Ouellet.

So it turns out that libcrypto on i386 platforms, unconditionaly compiles this
little gem called OPENSSL_indirect_call(), supposedly to be ``handy under
Win32''.

In my view, this is a free-win ROP entry point. Why try and return to libc
when you can return to libcrypto with an easy to use interface?

Better not give that much attack surface, and remove this undocumented
entry point.

ok beck@ tedu@

When compiling with AES_WRAP_TEST, make main() return a meaningful value
instead of garbage, and add this to the libcrypto regress. Note these tests
are incomplete, as they always use the default IV.

use reallocarray

Mention 16-bit sound is supported nowadays.

remove dead stores to error. from Fritjof Bornebusch

malloc/memset -> calloc. from peter malone

Remove the certs directory that contains ancient files not used
by anything.

"yes, blow it away" beck@

malloc/memset->calloc. with bonus null check. from peter malone.

null a pointer to prevent double free. from Dirk Engling

Remove files which look like actual code compiled in libcrypto, but isn't.
One even says (in comments): HAS BUGS! DON'T USE

fix memory leaks. from Dirk Engling

Remove meat which either duplicates code found in apps/, or is only of value
for 20th century historians, and can be put in the Attic.

malloc/memset -> calloc. from peter malone

Fix issue where we could jump into getdirtybuf without splbio() on a retry
that probably crashed espie.
ok tedu@