buffer_get_string_ptr's return should be const to remind
callers that futzing with it will futz with the actual buffer
contents

Improve error handling in dbopen().  If PRAGMA SQL statements fail,
report the error, close the database, and return failure from dbopen(),
such that the main program can recover and rebuild the database.

As noticed by stsp@, this can happen when database files are
accessible, but corrupt or in the wrong format, which will now
automatically be repaired.

Besides, use a safer idiom after sqlite3_open*() failure that also
handles out-of-memory situations correctly, and do not forget to
close the database after CREATE TABLE failure.

For LUNA88K, make sure the architecture name shown to the user
matches the architecture name required for man -S and apropos -S.
Adjust the case of LUNA68K to match the case of LUNA88K.
ok aoyama@

Regress for access()

typo in comment

Remove orphaned l2n6 and n2l6 macros.

Use C99 initializers for the various FOO_METHOD structs. More readable, and
avoid unreadable/unmaintainable constructs like that:

const EVP_PKEY_ASN1_METHOD cmac_asn1_meth =
        {
        EVP_PKEY_CMAC,
        EVP_PKEY_CMAC,
        0,

        "CMAC",
        "OpenSSL CMAC method",

        0,0,0,0,

        0,0,0,

        cmac_size,
        0,
        0,0,0,0,0,0,0,

        cmac_key_free,
        0,
        0,0
        };

ok matthew@ deraadt@

static const char * = "" -> static const char[] = "", to produce shorter code.
No functional change.

Unifdef -U OPENSSL_BUILD_SHLIBCRYPTO, since all it causes under Unix is to
redefine OPENSSL_EXTERN from `extern' to `extern'.

Put explicit (void) in function declarations and shuffle keywords in some
declaration to pass -Wextra, should we want to add it to CFLAGS.
No binary change.

Fix library search order in our libtool.

Before, libtool first searched all directories for .la files, and if
search failed, switched to actual libraries (.so/.a). But the correct way
is to check each directory first for .la, then for .so/.a.

The problem was reported by ajacoutot@ who verified that the patch fixes
his case and okay'ed the initial diff. And jasper@'s bulk uncovered a bug
which is fixed now (verified by another bulk).

zap trailing whitespace;

sort the ip6 sysctls;

We do not need a separate file for two compatibility wrapper functions.

ok miod@

Fix leak last commit introduced. Spotted by Sebastian Kapfer.

Document net.inet6.ip6.dad_pending.

OK from jmc@, and thanks to sthen@ for pointing out my mistake in the first
version of the diff.

zap a ton of silly Tn, and uppercase TTL; a few other minor tweaks along the
way;

Add PME_STATUS and PME_EN bitmasks for the PCI PMCSR register.
ok kettenis@

Start de-obfuscating the HC drivers.

Since pipe methods have an xfer argument, always use it to get acces to
various data structure (pipe, bus, device) instead of having a different
way to get a pointer to these descriptors in every function.

Also kill the {E,U}XFER() macro and use a consistent name accross all the
methods for {e,u}hci_xfer.

Don't zap disklabel (and next 6 blocks) when installing boot block on
4k-sector disk drives.

Noted and fix tested by David Vasek via misc@

tweaks & ok jsing@

Do not let the bluetooth HID device, present in Apple machines with USB,
grab the console.

This makes ddb(4) usable with any simple wired USB keyboard on G3/G4
machines that do not have adb(4) input devices, on G5s and certainly on
some MacBooks too.

ok andre@, miod@

Get back "_ppp" user and "_ppp" group.  From now they will be solely
used by npppd.

ok deraadt

transplant hotplug code over from qle(4)

sync

When the firmware has already allocated the handle we want to use to log in
to a port, restart the update process to keep things in sync.  Domain
controller logins get special handling here because they don't show up in the
list of logged in ports.

Appease gcc3 -Wuninitialized.

Get rid of duplication of the global lock code.  Allow recursion in all
code paths.

ok pirofti@

Replace all use of ERR_add_error_data with ERR_asprintf_error_data.
This avoids a lot of ugly gymnastics to do snprintfs before sending the
bag of strings to ERR, and eliminates at least one place in dso_dlfctn.c
where it was being called with the incorrect number of arguments and
using random things off the stack as addresses of strings.
ok krw@, jsing@

sync

Do not pass a NULL argument to OF_getprop(), even if it's fine for the
moment since we use an intermediate buffer and we specify a length of 0,
there's no need for an exception here.

Noticied while playing with the virtual-mode client interface of firmware.

Allow to compile with DEBUG_PMAP defined.

More KNF.

KNF.

Kill the link to /usr/bin/page.  OK henning@ jmc@ miod@ schwarze@

fix whitespace

Only remove queues from the TAILQ when the ticket changes; fixes PPS/BPS
displays for queues in "systat q". From Arto Jonsson.

This is not FreeBSD.

ok miod@

Do you really want to build OpenSSL for 16-bit Windows? Well, we don't.

ok miod@

Link traceroute6 to traceroute, thus completing the merge.
I was originally sent on this little errand by deraadt@.
OK sthen@, benno@
(Committing from a bus somewhere in germany on its way to berlin.)

remove a duplicate break statement

Revert 1.11. That patch is not only not necessary, it is quite harmful now.

ok schwarze@

Perform MAXPHYS-sized reads for the chunktable instead of PAGE_SIZE-sized
reads.

Also fix a VA leak in the chunktable read error path.

discussing terminfo is not needed; ok schwarze

This manual was lacking it's most important section, HISTORY.
While here, add .An (author name) macros.

Security is square, gaming is hip,
so extend the Valhalla Rampage to where it really matters.

Tedu 118 lines of dead code containing 77 dead magic numbers,
rotting in the tree since June 24, 2001.

Spotted by and ok jsg@, ok sthen@.

filter excess data from autoinstall output *before* it ends up in the
log file on the ramdisk, in order not to run out of its precious space

reported by, tested and ok sebastia@

Make more(1) POSIX compliant with respect to the -e option.
By default, "more" now works like "less -e", and "more -e" now
works like "less -E", and more never exits just because the file
fits on the first screen, like "less -F" would.

While documenting this, add some text to the DESCRIPTION explaining
how paging works.  That allows a very concise description of -e.

code change ok millert@
manual page joint work with and ok jmc@

Redirecting stderr to /dev/null suppresses all errors.  Instead use
the new status=none feature to make dd quiet.
OK halex@

add traceroute6 to NAME, and knock out some unneeded Bk/Ek;

sort; ok blambert

KNF.

Defer the free of the hibernate structures until later, when it is safe to
do so.

Freeing the hibernate structures too early leads to inadvertent calls into
splx when we aren't ready for them yet. Found when trapping splx calls in
the resume path.

discussed with deraadt@ at m2k14

Fix a minor optimization i broke in bsd.lv rev. 1.163 on August 20, 2010:
Do not bother looking into the hash table when the length of the macro
already tells us it's invalid.  No functional change.
Noticed by jsg@, thanks!

Fold in the zlib_cleanup macro.

do_pipe_sig is now only used in openssl.c - just call signal() directly
from there instead and ditch the macro.

NON_MAIN no longer means anything so remove it, along with APP_INIT.

ok miod@

The apps_{startup,shutdown} macro mess is now only used in openssl.c -
reduce the macro to two functions and just call them instead.

ok miod@

Merge Less 458 plus local patches

ok guenther@, ok jmc@ for man bits

Import of Less 458

ok guenther@

Clean up the remaining TTY related remnants.

ok miod@

specifying ECHOCTL once is enough
ok nicm@

regen

Add a bunch of GeForce GT{,X} chips listed in nvidia's 334.21 driver.

This includes the GeForce GT 740M submitted by Rafael Neves on tech@,
thanks!

kill dugsong's debugging leftover; ok sthen, jsg

Reduce the verbosity of makewhatis -t:
In the past, it always showed the title lines of the files processed.
Now, it only shows them when called with -D.
That is better because pkg_create calls makewhatis -t.
It is also more consistent with -D behaviour in non- -t modes.
Issue reported by ajacoutot@; ok espie@ ajacoutot@ jasper@.

MLINKS traceroute.8 traceroute6.8
"sure" jmc@

Merge traceroute6 man page with traceroute.
"sure" jmc@

Sync "struct rt_addrinfo" definition after removal of rti_ifp and kill
a trailing whitespace pointed by jmc@.

Remove NULL pointer dereference, which jsg insists I not
call a use-after free, from error path.

found by and ok jsg@

Remove rti_ifp from rt_addrinfo, one less ifp pointer, say yay!

This pointer was only needed by rt_getifa() to find an address, so
turn it into a local variable.

ok henning@, bluhm@

Do no reprobe for a supported protocol when enabling a pointing
device of type PMS_STANDARD, probing during autoconf(9) is enough!.

In such case, the device can be a simple PS/2 mouse, a unsupported
touchpad or a downgraded touchpad if something bad happened.  But
it is very unlikely that reprobing will improve the situation and
transform a standard mouse into a multitouch-aware touchpad ;)

This fixes the 12 seconds delay seen on various Dell laptops (E4310
and Latitude D630) when starting Xorg, reported by Kārlis Miķelsons
on bugs@.

ok stsp@, dcoppa@, shadchin@

test dt_min for invalid minute values instead of testing dt_sec twice
ok bmercer@

Kill in_localaddr(), one less usage of the global list of IPv4 addresses.

This function is used only once in our tree to optimize the size of the
MSS if the forward address correspond to a host on one of our subnets,
but only if ip.mutdisc is disable, which is not the default!

While here get rid of the "#ifdef RTV_MTU", it is here.

ok henning@, mikeb@, bluhm@

don't access a pointer till after the null check
ok mikeb@

Support running user-defined actions on receipt of snmp traps.

testing sthen@
much man page guidance jmc@
ok reyk@

be a bit clearer about what this page is; ok millert schwarze

Remove defined varibles that no longer do anything.

ok miod@

Use const char *.

Suggested by miod@

KNF fixes

Try to clean the maze of <openssl/bn.h> defines regarding the BN internals.

The intent of this change is to only keep support for two kind of architectures:
- those with 32-bit int and long, and 64-bit long long, where
  ``long * long -> long long'' multiplication routines are available.
- those with 64-bit int and long, and no 128-bit long long type.

This gets rid of the SIXTY_FOUR_BIT_LONG, SIXTY_FOUR_BIT (not the same!),
THIRTY_TWO_BIT, SIXTEEN_BIT and EIGHT_BIT defines.

After this change, the types and defines are as follows:

arch:           64bit           32bit           rationale
BN_LLONG        undefined       defined         defined if l * l -> ll
BN_ULLONG       undefined       u long long     result of BN_LONG * BN_LONG
BN_ULONG        u long          u int           native register size
BN_LONG         long            int             the same, signed
BN_BITS         128             64              size of 2*BN_ULONG in bits
BN_BYTES        8               4               size of 2*BN_ULONG in bytes
BN_BITS2        64              32              BN_BITS / 2

Tested on various 32-bit and 64-bit OpenBSD systems of various endianness.

Re-add rcsid, and switch back to our previous descriptions for -iDsym[=value]
and -iUsym which jmc and I find a bit easier to understand. ok jmc

sync

Install separate more.1 instead of using a link from less.1.
Requested by jmc@

Fix indentation.

Bring back the SSL_OP_PKCS1_CHECK_* defines (defined to zero, as they already
were), after stumbling upon code (mis)using them in the wild.

rearrange a bit of code/comments

combine some notes in STANDARDS for consistency;

document the new makewhatis(8)/apropos(1) database format;
original idea to have some documentation about this by espie@;
ok jmc@

no more kinit(1);

jmc spotted more ruptime tentacles

uses ssh these days, not rsh; ok tedu

no more rwho/ruptime;

unifdef MSDOS
ok miod@

obvious use after free was less obviously not a use after free.
who uses '\0' as a null pointer constant? fix my mistake and restore
original semantics.

Fix logic inversion when checking environment variables on the
command line against the blacklist.  This is only a problem when
env_reset is disabled.  CVE 2014-0106

More KNF.

no need to check for default_chip_map twice in one if statement

Make it compile again.

Remove some malloc/strlcpy/strlcat horror. Just use asprintf and avoid an
unchecked malloc at the same time.

ok beck@

regen