yet more code that no longer makes any sense

makewhatis being now external, it no longer requires picky/test/check_dir.
It also can't handle too many manpages at once (shell limit) so hand-feed
it (problem spotted by landry@).

Some tests by schwarze@.

fix bogus extension: $code and $code2 belong in _system, so that
both system and verbose_system will work properly.

Use errc/warnc to simplify code.
Also, in 'ftp', always put the error message last, after the hostname/ipaddr.

ok jsing@ krw@ millert@

Bring UTF8_{getc,putc} up-to-date: it's been a decade since 5- and 6-byte
encodings and encoding of surrogate pair code points were banned.  Add
checks for those, both to those functions and to the code decoding the
BMP and UNIV encodings.

ok miod@

Format string fixes and removal of -Wno-format for sgi. Based upon an
initial diff from jasper@

X509_free() the certificate if the server hostname check fails.
No functional change.  Reported by Mike Small and Maxime Villard.

Fix several bounds checks in ssl_add_clienthello_tlsext() and
ssl_add_serverhello_tlsext(), and convert all of them to the same idiom, for
easier review.

Math is hard, let's go webshopping.

Help and ok guenther@

Explicitely initialize two static variables introduced in the previous
commit, to please lteo@

HTTPS connections may see redirects, so initialize libcrypto and libssl
only once, and reuse the crafted SSL_CTX for further connections.
ok lteo@

Plug a small memory leak: if we pass checkin_init/checkin_update
a revision thru pb.newrev, it will not be freed.  From Fritjof
Bornebusch, no objections from tech@

no more -K;

kerberos leftovers

consistent use of uint32_t for poison values

Add some consistency in fuse error handling. We need to return ENXIO
when the fuse communication channel is broken and ENOSYS when the
libfuse callback does not exist.

Reported by Helg Bredow, thanks!
ok tedu@

as reported by Helg, should use realpath for mounts. ok guenther syl

There is no need to remember which usb(4) device is the child of an USB
host controller because autoconf(9) already does it.

Breakage reported by todd@

sync

More KNF.

More whack-a-mole^Wfips.

ok miod@

It is perfectly valid to call free(NULL), so stop checking for non-NULL
before calling free - of some course parts of the code already did this.

Nuke some trailing whitespace that snuck in.

No point in trying to update the -L file on loss of link if no -L has
been specified. Eliminates spurious "Couldn't write ''" log messages.

Spotted while looking into Roman Gorelov's logs on another problem.

Add a H_SAVE_FP operation to history() which lets the history be saved
to an open file pointer. From NetBSD via Eitan Adler.

ok millert

fix format mistakes visible if AUDIO_DEBUG is defined

unbreak the build when DIAGNOSTIC is not defined
allows ramdisk kernels to build again

Update run(4) firmware to version 0.33, with permission from Dennis Lee
of Mediatek to keep using the free firmware licence we've been using.
Tested by myself, rapha@, Fred Crowson, Dinar Talypov, gerhard@, matthieu@.

Enable the 3- and 4-byte sequence tests for UTF8_getc()
Add surrogate and out-of-range tests for UTF8_putc() on the assumption
we'll make it return -2.  Maybe.

Add regression test for UTF8_{getc,putc}()

hid_start_parse can't fail - it calls malloc(M_WAITOK).  ok mpi@

Correctly handle files smaller than an ELF header; reported by drahn@bitrig

_PIC is an optional method, so we shouldn't be aborting the rest of the
MADT processing if it is missing. This, combined with an earlier diff
from kettenis@ to fix MADT interrupt polarity processing, should resolve
the issues reported with QEMU/KVM ACPI power button presses when using
OpenBSD guests.

ok kettenis@, pirofti@
tested by myself and kettenis for the past few weeks

If you need to allocate `a + b' bytes of memory, then don't allocate `a + b*2',
this is confusing and unnecessary.

Help (coz I got confused) and ok guenther@ beck@

Make sure UTF8_getc() is invoked with the proper buffer size.

ok beck@ guenther@

use reallocarray
okay ian@, chl@

factor out a bit of the chunk index code and use it to make sure that a
freed chunk is actually freeable immediately. catch more errors.
hints/ok otto

Do not pass an xfer pointer to the timeout routine checking for root hub
status changes because it might be freed when detaching the root uhub(4).

Also do not reschedule a timeout if the pipe is being aborted.

Finally do not add more code to retrieve the 'bInterval' value of the
root hub endpoint descriptor since this value is hardcoded in the uhci(4)
driver.

Out of all the openssl apps, just one of them has code to mess around with
electric fence externs. Nuke from orbit.

ok miod@

use reallocarray
okay miod@, millert@

W_OK comes via unistd.h - nuke several #ifndef W_OK that attempt to handle
missing defines.

Remove unused include.

Group and sort includes.

Currently "apps.h" needs to be included before any of the openssl headers
and there are several cases where the code depends on headers that are
included by something included by apps.h...

More untangling saved for later on.

Plug memory leaks upon error in OCSP_url_svcloc_new().

ok jsing@

In OCSP_sendreq_bio(), cope with OCSP_sendreq_new() returning NULL.

ok jsing@

In dtls1_reassemble_fragment() and dtls1_process_out_of_seq_message(), in case
of error, make sure we do not free pitem which is still linked into the
pqueue.
In the same vain, only free `frag' if we allocated it in this function.

Help and ok beck@

In ssl3_send_certificate_request(), when adding the extra payload if
NETSCAPE_HANG_BUG is defined, make sure we BUF_MEM_grow() the buffer to
accomodate for the payload size.

Issue reported by David Ramos; ok beck@

Make sure ssl3_setup_buffers() does not return upon error with a freed
pqueue still chained, by inserting it into the list only after all possible
failure conditions have been avoided.

Reported and fix proposed by David Ramos; ok beck@

Fix tail packet check in elantech v3 touchpad code. Due to a typo this
code was masking out bits which were also tested in the same expression.
Reported by cppcheck via jsg@ (Expression '(X & 0xfc) != 0x2' is always true).
Tested with Elantech Touchpad, version 3, firmware 0x250f00.

Don't add a lease to the leases TAILQ more than once. It tends to
make writing out the leases file go into an infinite loop until
/var is full.

Reported by Roman Gorelov via bugs@

Fix tested & ok stsp@ uwe@

sync

bio_err is initialised in main, hence there is no need to initialise it
in each application (since there is no longer a non-monolithic mode).

In typical OpenSSL fashion, the code is inconsistent and there are multiple
ways that bio_err was initialised - none of them actually checked to see if
the initialisation actually succeeded. Additionally, it is worth noting
that in at least two cases bio_err was also being used before it would have
been initialised.

ok miod@

More KNF.

Omit -Werror if building with gcc3, for it triggers "redefinition of `pqueue'"
warnings now that pqueue.h has been removed from public scope.

Make rtsx(4) attach to RTL8402 devices. Reported as working by Eric Huiban.

regen

Add PCI ID for Realtek RTL8402 card reader.

No need to check for NULL before invoking free(); use calloc() when
applicable; further simplify pqueue_find().

From Dimitris Papastamos on tech@

use reallocarray and friends. okay miod@

sync openssl(1) with changes already made in src/lib/libssl/doc, reminded by
jmc: move from 1024 to 2048 bit key defaults; see genrsa.c 1.26 et al.

use reallocarray
okay chl@, tedu@

a bit more reallocarray (and kill ecalloc, which isn't used)
okay chl@

There is no need to compile nginx with -O1 on vax anymore.
Since the change in rev 1.5 vax has switched to gcc3,
MAXDSIZ has increased and pcre_exec now builds with -O2.

ok miod@

replace some (type *)0 to NULL

ok krw@ millert@

One more mistake introduced in 1.65

remove unused variables

ok miod@

...and it needs a guard against NULL as well.

Fix previous commit by making sure we are not dereferencing uninitialized
pointers.

remove unused variable

ok tedu@

does sunos not have memmove? sorry, i guess sunos will not have libressl.
ok guenther miod

add missing header needed by str* and mem* functions

ok gilles@

fix alloc idioms. watch out for emalloc that's actually a calloc in disguise
okay chl@ tedu@

unchecked malloc() return value in OCSP_sendreq_new().

KNF

add missing header needed for mem{cpy,cmp}()

ok tedu@

correctly match size and buffer. from enh at google

simple reallocarray to check multiplies. ok guenther

When looking at another process, use the data from struct process
instead of peeking into the thread level

ok tedu@

no no md2

no no seed

deleted seed some time ago

remove unused variable

ok tedu@

Add missing header needed by isdigit().
While there, sort headers.

ok tedu@

refactor some of the des crypt code to use fewer globals.
ok miod

Fix kernel build with EHCI_DEBUG by removing a non-compiling DPRINTF().
Since no DPRINTF() call remains also remove its definition (requested by mpi@).
ok mpi, and also reported by Jean-Philippe Ouellet

fix format mistakes visible if AZALIA_DEBUG is defined

fix format mistakes visible if UAUDIO_DEBUG is defined

fix format mistakes visible if AUICH_DEBUG is defined

unbreak build with AUDIO_DEBUG: use %lx for bus_addr_t

In pmap_remove_page(), do not keep wired mappings; the callers want them gone.

Remove "WARNING:" prefix from SYSFAIL and ACFAIL messages, so that, in case
of ACFAIL, what reaches the console is explicit enough.

unbreak build: exchange %b args order, from miod@

Lower VUPS value for delay() to behave closer to the intent.

This is temporary until vax grows code to compute a proper delay const
at runtime, but will help a bit, especially on KA46 where cd(4) end up
detected as sd(4) due to too short delays.

switch RSA key generation default to 2048 bits (matching ssh); ok miod@

Print the rpb device type and unit values if the root device can't be found
by the kernel.

When the firmware tells us a loop id is already in use, add the device
there to our port lists, then find the next available loop id and carry on
rather than restarting.  This way, we're less likely to get stuck looping
when the firmware behaves inconsistently.  We also don't need to treat domain
controller logins specially.

additional complications discovered by deraadt@

Change the return value of interrupt handler to 0 when nothing is
processed on this driver.

"This is correct" miod@

When parsing a numerical value for the TOS bits, make sure that it
is in a valid range.
OK henning@

better description for -A and -a, with help from millert;
ok millert sobrado guenther

update NAME and MLINKS for previous;

more STANDARDS fleshing; ok sobrado millert