mkdep(1) seems to want the current directory on the include path,
so reinstate it in the Makefile.

Reported-by: drhodus

Now that we have dhcpd and dhcrelay in base, fix the inherited rcNG scripts
and install them.

Raise WARNS to 6:
- Initialize a variable that GCC can't tell will never be used
  uninitialized.

Raise WARNS to 6:
- Use a fully-specified NULL-filled entry to terminate an array
  initializer;
- Remove an inner-block redeclaration of a variable with the same
  name and purpose as the local in the outer block.

Raise WARNS to 6.

Advised-by: joerg

Reduce warnings when compiled under WARNS=6:
- Constify many char *'s;
- Rename a local variable;
- Initialize an option list element using strdup (as is done elsewhere)
  instead of directly assigning a string constant to it.

Correctly reset place in getopt_long_only the next time it is used.
Remove unused retval in getopt wrapper.

Raise WARNS to 3:
- Make an int variable unsigned;
- Use WARNS level instead of hand-specifying warning flags;
- Remove unneeded -I. -I${.CURDIR} from CFLAGS in Makefile.

Raise WARNS to 6:
- Constify several char *'s.
- Mark unused parameters as __unused.
- Rename local variables.
- Format function prototypes and remove parameter names.
- Expand a macro that was only used twice (on the same
  parameter) into a real variable to eliminate signed
  versus unsigned comparison warning [1].

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>
Additional-work-by: cpressey [1]

Don't write the name of the sysctl to a temporary variable and truncate
it on the '=' there. If the argument is actually longer than the buffer,
this would result in a NULL reference. Instead, allocate the necessary
buffer with alloca and copy the name part.

Discovered-by: Suleiman Souhlal <ssouhlal@FreeBSD.org>

unknown_incoming may be customized so we only install it if it doesn't
already exist.

Noticed-by: Joerg Anslik <joerg@anslik.de>

- Add WARNS?=6 and $DragonFly$ to Makefile.

- Add back marco that joerg added that I deleted, because I thought
  I added it by accident. :-)

Scheduled mdoc(7) sweep.

Taken-from: FreeBSD
Author: ru

- Clean up wc (remove (void) etc)
- Remove unnecessary cast
- Add WARNS 6

Submitted by: Alexander 'alxl' Lobachov <alxl@alxl.info>

Add some documentation for ntpd_flags.

- Remove unnecessary headers
- Remove main proto
- Constify
- Add WARNS 6
- Minor cleans

ntpdate(8) is gone and has been replaced by rdate(8).

Add default if_re_load line.

typo

Add gmp handling to upgrade_etc.

Remove LIBGMP and LIBMP.

Deorbit libgmp.

Convert to keyserv, telnetd and telnet to libcrypto's BIGNUM
implementatation.

Remove another Perl left over.

Submitted-by: Chris Pressey <cpressey@catseye.mine.nu>

Remove some debugging printfs. Fix an error message.

- Swap location of Buf_Expand() function.

- Pass NULL as 2nd param to Buf_GetAll(buf, &len), the callers don't need
  that info.

Add forgotten cast back. This code depends on BASE being short and
signed.

Switch chkey/newkey to use libcrypto instead of libmp for the internal
PK generation.

Submitted-by: Dmitri Nikulin <setagllib@optusnet.com.au>
Additional cleanups and bugfixes by myself.

WARNS=6 cleaness.

- Inline Buf_OvAddByte() since it is only used in one place.

Make newkey WARNS=6 clean.

Mark OpenSSL 0.9.7d as dead.

Add splitpatch.

Add local_syms script. Nice for cleaning the kernel namespace.

Update OpenBSD tags to reflect which revisions we are synced with.

Raise WARNS to 6 and generally clean up:
- Constify and use size_t instead of int;
- Merge and add variables as appropriate;
- Use more appropriate printf() format specifier, and replace
  printf() with fwrite() in one instance;
- Replace name2oid() with sysctlnametomib().  Retain old error
  message when the sysctl's name cannot be resolved to a MIB.

Based-on-coincident-patches-by: myself, and Peter Schuller
                        <peter_dot_schuller_at_infidyne_dot_com>
Refined-with-input-from: joerg
OK'ed-by: joerg

Remove perl's man path.

Remove documentation of NOPERL, it's the default now. Don't create an
include hierachie for perl.

Remove wx(4) man page as well.

Remove wx(4). It's been superseded by em(4).

- Nothing modifies outPtr in the Buffer object.  Since it is set to
  buffer when the object is created, replace references to it with
  buffer, and remove outPtr field.

Better diagnostic for getopt_long_only.

Fix two small bugs in getopt_long_only handling:
- a short option followed by a long option is parsed as that
  (inspired by OpenBSD)
- the place variable has to be reset after long option parsing failed

Add support for getopt_long_only. It allows using long options with a single
dash by prefering them over aggregated short options.

- Move code into buff.c by converting #define macros to real functions.

Move the handling of '--' as argument into getopt_internal. Add a parameter
to getopt_internal and getopt_internal_short to activate support for long
options. Disable the special 'W' handling for getopt(). Correctly handle
long option like arguments ('--l') as arguments '-' and 'l'.

Move short option parsing into its own function. Constify the local char *
pointer.

- Replace Buf_Discard(buf, size) with Buf_Clear(buf).  This simplifies
  the code, and make clear what is taking place.

- clean up a leftover in binutils215/ld
- rm: /usr/libexec/binutils214: is a directory

Noticed-by: David Rhodus

- Last of the easy constifications.

- Constify more functions.

- Cleanup white space. style(9)

RIP Perl.

- Moved unused argc, temp variable into small scope.

Increase WARNS to 6

- Move sh_builtin string array into shellneed().  It is the only
  function that uses it.

- Correct comment describing shellneed() function.

- Since shellneed() can never return -1, remove code that checks
  for it in the caller.

- Add WARNS 6 and make WARNS 6 clean
- Remove registers
- Move global variables into main()
- Static utmpcmp()
- Remove cast in switch()
- Remove duplicate utmpcmp prototype
- Minor cleanup (ANSI, remove (void))
- Add $DragonFly$ tag

Submitted by: Richard Bradshaw <theglassishalfempty@gmail.com>

Don't create binutils 2.14 and perl directories. Remove perl and
binutils214 [later only, if gcc34 and binutils215 are installed].

It's WARNS?=6, not WARNS=6.

Noticed-by: corecode.

- Add WARNS 6

- Add -k option for whois.krnic.net (hold details of IP address
  allocations within Korea) - FreeBSD
- Remove unnecessary casts
- Constify
- Update man page to reflect new option (-k)

Raise WARNS to 6:
- Add function prototypes.
- Rename a parameter.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>

Add /sbin/mountctl to the build.

Revamp the argument format a bit, add basic suppot for creating, deleting,
and listing journals.  None of the option features can be specified yet,
and the journaling code is only logging a few VOPs at the moment for testing
purposes, but you can actually create a journaling stream now.

Add support for retrieving the journal status via mountctl.  Increase some
of the buffer limits.

This is the initial skeleton for the new mountctl utility.  Manual page,
options parsing, sanity checks, and the scan-callback framework.

Constify more functions.

Constifying functions.

ANSIfy. Fix various warnings. Statify.

Submitted-by: Peter Schuller <peter.schuller@infidyne.com>

Constify more functions.

Further cleanup:
- When fs_Bytes is zero, assign NULL (rather than "") to fs_Data.
- Check that fs_Bytes is nonzero before subsequently using fs_Data.

Requested-by: dillon
Tested-by: typical usage of installer

Don't clobber match_str in buildmatch by strdup'ing the string internally.

Submitted-by: Peter Schuller <peter.schuller@infidyne.com>
Additionally remove func_name define, use __func__ instead.

Fix a memory leak in regex.

Submitted-by: Steve Mynott <steve.mynott@gmail.com>
Taken-from: http://www.freebsd.org/cgi/query-pr.cgi?pr=75656

Fix one of probably several smbfs issues.  smbfs is improperly tracking
open() and close() calls and closing the remote FID on the last close().
This appears to be done due to the limited number of FIDs available.
The problem is that the vnode might still be referenced and used by mapped
memory (mmap), so smbfs_getpages and putpages will seg-fault the user
program if there is no open descriptor on the file.

For now temporarily reopen and reclose the file.  This is inefficient, but
it should work.  The whole tracking section of smbfs really needs a rewrite.

There are guarenteed to be many other issues with smbfs.

Reported-by: Rumko <rumcic@gmail.com>

Add missing #include

Constify more functions.

Do not specify the -B option when executing the sub-make.  In the BSD
make, -B is a compatibility option to execute shell commands one per line.
In GNU make (as of vers 3.80) -B is --always-make, which forces all targets
to be made regardless of whether they are out of date or not.

Since both GNU make and our make use MAKEFLAGS to pass build flags between
invocations, -B gets propogated down the chain into the gnu make and blows
everything to itsy bitsy little pieces.

Constify more functions.

Increase the default TCP maximum segment size from 512 to 1460.
The modern consensus is that 1500 is a safe MTU to use for the Internet.

Remove the sysctl options for altering the initial TCP congestion window size.
They violate the TCP spec and don't really buy the user anything, especially
now that we have RFC3390 support for larger initial window sizes on by default.

Remove the sysctl options for altering the initial TCP congestion window size.
They violate the TCP spec and don't really buy the user anything, especially
now that we have RFC3390 support for larger initial window sizes on by default.

Fix copyright notice.

Same as FreeBSD rev 1.9 to uthread_mattr_init.c and rev 1.8 to
uthread_mattr_kind_np.c.

While removing a memory leak, rev 1.32 introduced a
free-memory-and-reuse-it-after. Correct both problems and
make rcp -r work again under /etc/malloc.conf -> AJ.

Submitted-by: Xin LI <delphij@frontfree.net>
Taken-From: FreeBSD/1.37 charnier
PR: FreeBSD:bin/54969

Correct two bugs that may result in incorrect CBCP response for
administrator specified number.

The implementation in our version transmits extra zero byte that is a
violation of the Microsoft PPP Callback Control Protocol section 3.2[1].
This can lead to incorrect behavior on certain devices, as reported
in bin/50613.

Confirmed with: cbcp.c in ppp's cvs, rev. 1.4 and 1.15 [2]

Submitted-by: Xin LI <delphij@frontfree.net>
Taken-From: FreeBSD/rev1.7
Submitted-to-FreeBSD-by: Dmitry Pryanishnikov <dmitry at atlantis.dp.ua>
PR: FreeBSD:bin/50613

Merge ... FreeBSD rev. 1.36+1.37 of ip_nat.c.  Conditionalize declarations
properly when LARGE_NAT is not defined.

Submitted-by: Xin LI <delphij@frontfree.net>

Change variable name from retstr to ret. Reduce diffs between FreeBSD

Catch up to recent rtlookup() changes.

Submitted by: Tim Wickberg <me@k9mach3.org>

Raise WARNS to 6:
- Constify a string variable;
- Parentheses around an assignment in a test.

And style(9):
- Remove casts to void when ignoring return values;
- Remove `register' keywords;
- Make check against '\0' explicit.

Raise WARNS to 6:
- Make an extern declaration global.

And style(9):
- Remove `register' keywords;
- Braces around multiline blocks;
- Remove casts to void when ignoring return values;
- Make tests against NULL and 0 explicit;
- Untangle a particularly convoluted conditional printf().

Add missing prototype that I missed with spliting up nonints.h

Clarify/clean up code, make fewer assumptions about types:
- Store pids/gids/uids/devs in a union instead of casting to a long;
- Don't use the nonspecific LT_GENERIC list type;
- Add a more specific LT_PPID list type for parent pids;
- Use NODEV to indicate the lack of a specified device;
- Make the makelist() function more straightforward:
  - use only one switch statement instead of two cascading ones;
  - introduce a helper function parse_pid() to parse pids;
- No functional changes.

Fix a kernel crash that occurs when the SMB protocol stack is used.  The
SMB protocol stack is still hacked to create a kernel thread with a
process structure attached to it (when we really ought to be using a pure
kernel thread... this is the only subsystem that still doesn't).  However,
the SMB code was not properly cleaning the process structure up on exit
resulting in a corrupt process a list.  A 'ps' equivalent occuring after
this situation occurs can crash the kernel.

Reported-by: Rumko <rumcic@gmail.com>

enomem() is only used in one file, make it static.

Removed unused routines:
Buf_GetByte()
Buf_GetBytes()
Buf_UngetByte()
Buf_UngetBytes()

Style(9):
- braces around multiline blocks
- no blank line at start of function
- remove extraneous semicolon.

Now that I understand the poorly written BSD routing code and what
it was trying to do, rewrite it in a clear and concise manner.

The old rtalloc1() code written by CSRG had a number of problems:
  1. it was not clear which route was being returned
  2. it was not clear what was being reported
  3. it hid the essential radix tree lookup operation inside a series of
      conditional tests and inline assignments
  4. it had multiple gotos to the inside of if statements
  5. it intermixed reporting code with the operational logic of lookup
      and cloning
  6. it assigns multiple times to key variables
  7. it has unnecessary assignments to key variables
  8. it overloaded the "report" argument parameter, to have two
      different semantics
  9. it misnamed the key route lookup function "rtalloc1", obscuring all uses
      of route lookup.

In contrast to the rtalloc1 code in FreeBSD 4 or the even more convoluted
rtalloc1 code in FreeBSD 5, the DragonFlyBSD version
  A. has a clear control flow that makes the common case obvious
      by highlighting the core call to the radix tree look up function,
      eliminating gotos into if statements,
      and completely separating out the special-case cloning logic
  B. makes it clear which route is being returned
      by only assigning once to the key "rt" variable
      and by expliciting returning "rt" or "clonedroute"
  C. abstracts out the reporting code into its own reporting API
  D. cleans up the semantics of the "report" argument parameter to
      only indicate whether to report a miss and not whether to clone
  E. introduces a simple single-argument API for caller that want to clone
      and those that do not.

Fix compile error.

Move a global variable into local scope for MP safety.