KNF.

Nuke pointless dev_bsize variable which is always set to 512, and just
use DEV_BSIZE.

Replace lseek+read with pread().

update nosuidcoredump example. mentioned by stefan wollny

Nuke pointless dev_bsize variable which is always set to 512, and just
use DEV_BSIZE.

Write sizes need to be secsize and not DEV_BSIZE, just like reads.

Equivalent to changes made in fsck_ffs.

More KNF.

Make this build cleanly even with UTF8_{putc,getc} moved from the
public header to the internal.  Make some variables unsigned to
eliminate some warnings

No uses of UTF8_{getc,putc}() in ports, so remove them from the public header.

ok miod@

Nuke some trailing whitespace to shrink subsequent diffs.

From FreeBSD: the second argument to UFS_UPDATE/ffs_update is just a
boolean, not a MNT_* flag.

ok beck@

We only have one possible location to use for a logged in port, so only set
it in one place.

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.

Nuke some trailing whitespace to shrink subsequent diffs.

stat(2) on the spool file needs privileges.  Fixes file size reported by
lpq -l.

ok millert@

Nuke some trailing whitespace to shrink subsequent diffs.

Nuke some trailing whitespace and break some long lines to shrink
subsequent diffs.

Nuke some trailing whitespace to shrink subsequent diffs.

Fix 'fdisk -u', a.k.a. MBR_pcopy(), on sparc64 and possibly others.

gcc's built-in/in-lined memcpy() can't copy elements of a struct
dos_partiton array from inside the __packed struct dos_mbr into an
'unpacked' local variable. But copying the whole array works fine. Be
consistant and use the same idiom in both places the partition table
is parsed.

Leave the landisk workaround/hack in place until it can be confirmed
it is no longer needed.

ok martynas@ after many alternatives crashed and burned.

fix MALLOC_STATS (not compiled in by default); ok tedu@

Fix IPsec route addition broken since the removal of the link-layer
addresses from the per-ifp list.

While here document why enc(4) needs a link-layer address, or at
least something that seems to be one.

Found the hard way and fix tested by naddy@, ok mikeb@, henning@

regen

Add VIA Labs.

Handle the stall condition just like the bable one since in both cases
the ring is halted.

Do not bother reporting USBD_STALLED to the stack like other HC drivers
do since the endpoint is automatically reseted.  What is the point of
this error apart from making sure driver authors will forget to call
usbd_clear_endpoint_stall_async() correctly?

The Renesas uPD720202 xHCI, provided by Stefan Wollny, now works as
expected.

when failing because of unexpected output, show the offending output

Use errc/warnc to simplify code.

ok jca@ krw@

document which keywords are posix conformant;
ok guenther sobrado

Exit with non-zero status if a read is truncated, or if a compression
program was used but it didn't exit successfully.

Original diff by mpf@

Change use of Z_PARTIAL_FLUSH to Z_SYNC_FLUSH in the hibernate code as per
a warning/comment in zlib.h that Z_PARTIAL_FLUSH "will be removed, use
Z_SYNC_FLUSH instead".

Typo in comment noticed during some recent hibernate debugging. No code
change.

Fix invalid / missing format specifiers in several dnprintf statements
(used only when ACPI_DEBUG is enabled)

From a diff on tech@ by Fabian Raetz, with a minor change by me, tested
on i386 and amd64 with ACPI_DEBUG enabled and disabled.

warn about implicit. misspelled function names in libc are deadly.

destinct is spelled distinct.

ok henning@ jmc@

remove irrelevant, outdated, and misleading sentence suggesting that
you need to set up a local mirror for the install sets

brought to attention by pascal@ ok krw@

Merge the fixes from the upstream.  Factor out some common code in
tty.c and fix EL_SETTY to work.

ok nicm

As suggested by guenther@ and millert@, replace seek+[read|write] with
p[read|write].  Makes the code much clearer by eliminating extra error
checking and verbiage.

No intentional functional change.

Tweaks by and ok guenther@

format string cleanup: change "%i" to "%d" and fix a few typos

Unify the SSL privsep key loading functions.

ok eric@

KSSL is dead... nuke KSSL_DEBUG from orbit.

ok beck@ miod@

regen

Add VIA VL80x xHCI.

Format string fixes for XHCI_DEBUG.

Deep down inside OpenSSL, err... LibreSSL, RSA_set_ex_data attempts to
free() the external data when releasing the RSA object.  The
RSA_GET_EX_NEW_INDEX(3) manual page doesn't mention that this is the
default behaviour - it just describes the possible free_func()
callback - and the code path in libcrypto is hiding the fact behind
layers of abstraction.

Fix possible double free by allocating and copying the external data
reference that is used for RSA privsep (pkiname in smtpd's case).

ok eric@ gilles@

Add -o max_read=XXX support in fuse. This is needed by usmb to have a
working read() and write().

ok tedu@

Add support for -o XXX or -oXXX options in libfuse.

inputs from tedu@, ok tedu@

The argv[] used for el_set() EL_BIND, EL_SETTY, etc should be terminated
by NULL. Fix from NetBSD via yasuoka@ but changed by me not to walk off
the end of array with >20 arguments.

ok yasuoka

Assign the queue ID to the correct packet header when sending ICMP messages.

ok henning@

Fix eui64 address generation, broken since the removal of the link-layer
address from the per-ifp list.

Found the hard way by weerd@, florian@ and stsp@, ok florian@

correctly match size and buffer. from enh at google

remove dead files

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.