Add recommendation to use gpart(8) when user tries write disklabel
or bootcode to already opened provider.

MFC after: 1 week

Partially revert r227233.

The privs.h header is not only used by at(1), it's also used by
atrun(8). Just let the code the way it used to be (for now).

Reported by: kwm, tinderbox
Hat to: me

Remove MALLOC_DECLAREs of nonexisting malloc-pools.

After careful grepping, it seems none of these pools can be found in our
source tree. They are not in use, nor are they defined.

Initialize all components of the sent COOKIE.

MFC after: 3 days.

Remove unneeded checks.

MFC after: 1 week

Mark global functions and/or variables in spray(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in snapinfo(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in rmt(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in rip6query(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in pwd_mkdb(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in moused(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in mixer(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in memcontrol(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in fdread(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in devinfo(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in chown(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in boot0cfg(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Mark global functions and/or variables in apm(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

bsdlabel(8) could automatically fill many of disklabel's deprecated
fields, but user could specify some of those fields when edits disklabel
with `bsdlabel -e`. But without -A flag these fields might be
overwritten with default values from the virgin disklabel.
So, don't overwrite such fields if they are not zero. Also add checks
to prevent creating disklabel with less than DEFPARTITIONS and more
than MAXPARTITIONS partitions.

PR: bin/162332
Tested by: Eugene Grosbein
MFC after: 1 week

Mark global functions and/or variables in xstr(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in whois(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in which(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in whereis(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in unifdef(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in paste(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in m4(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in lam(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in fstat(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in expand(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in enigma(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in compress(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in comm(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Mark global functions and/or variables in checknr(1) static where possible.

This allows compilers and static analyzers to more thorough analysis.

Add missing static keywords to at(1).

While there, tidy up the privs.h part, where at.c has to #define to
declare some globals. Also group static and non-static global variables
in at.c.

To be in sync with GEOM_PART_BSD limit the maximum number of supported
partitions to 20.

MFC after: 1 week

Mark the structure fields as const.

This allows us to simply place these strings as constants in the table
itself, letting the compiler decide to simply merge duplicate strings.

Make all functions and global variables static for cdcontrol(8).

While there, replace __const by const, which seems to be our preference
nowadays. Also fix some style(9) bugs by adding newlines and removing
unneeded spaces from function declarations.

Mark all global variables static for mptable(1).

While there, remove basetableEntryTypes, which became unused in r71209.

Eliminate global variables.

There is no reason why these three variables should be declared as
global variables, while the others aren't. They are only used inside
main().

Simplify inclusion of the tsptype array a bit.

We don't need this array in timed.c -- only readmsg.c.

When one attempts to compile the tree with -march=i386, which also used
to be gcc's default before r198344, calls to atomic builtins will not be
expanded inline.  Instead, they will be generated as calls to external
functions (e.g. __sync_fetch_and_add_N), leading to linking errors later
on.

Put in a seatbelt that disables use of atomic builtins in libstdc++ and
llvm, when tuning specifically for the real i386 CPU.  This does not
protect against all possible issues, but it is better than nothing.

Add more obsolete files.

Cache SO_REUSEPORT socket option in inpcb-layer in order to avoid
inp_socket->so_options dereference when we may not acquire the lock on
the inpcb.

This fixes the crash due to NULL pointer dereference in
in_pcbbind_setup() when inp_socket->so_options in a pcb returned by
in_pcblookup_local() was checked.

Reported by: dave jones <s.dave.jones@gmail.com>, Arnaud Lacombe <lacombar@gmail.com>
Suggested by: rwatson
Glanced by: rwatson
Tested by: dave jones <s.dave.jones@gmail.com>

Before dereferencing intotw() check for NULL, the same way as it is
done for in_pcb (see r157474).

MFC after: 1 week

Fix the typo made in r157474.

MFC after: 3 days

Correct the change made in r227166.

This function is only compiled in when building WITH_AUDIT.

Add missing static keywords to xinstall(1)

Add missing static keywords to wc(1)

Add missing static keywords to wall(1)

Add missing static keywords to w(1)

Add missing static keywords to vis(1)

Add missing static keywords to uuencode(1)

Add missing static keywords to usbhidctl(1)

Add missing static keywords to usbhidaction(1)

Add missing static keywords to units(1)

Add missing static keywords to uniq(1)

Add missing static keywords to unexpand(1)

Add missing static keywords to uname(1)

Add missing static keywords to ul(1)

Add missing static keywords to tsort(1)

Add missing static keywords to tr(1)

Add missing static keywords to tee(1)

Add missing static keywords to tcopy(1)

Add missing static keywords to talk(1)

Add missing static keywords to tail(1)

Add missing static keywords to split(1)

Add missing static keywords to seq(1)

Add missing static keywords to rwho(1)

Add missing static keywords to rwall(1)

Add missing static keywords to rusers(1)

Add missing static keywords to ruptime(1)

Add missing static keywords to rs(1)

Add missing static keywords to quota(1)

Add missing static keywords to pr(1)

Add missing static keywords to mt(1)

Add missing static keywords to ministat(1)

Add missing static keywords to make(1)

Add missing static keywords to look(1)

Add missing static keywords to logger(1)

Add missing static keywords to lock(1)

Add missing static keywords to last(1)

Add missing static keywords to join(1)

Add missing static keywords to id(1)

Add missing static keywords to fold(1)

Add missing static keywords to du(1)

Add missing static keywords to dc(1)

Add missing static keywords to cut(1)

Add missing static keywords to csplit(1)

Add missing static keywords to cpuset(1)

Add missing static keywords to column(1)

Add missing static keywords to colcrt(1)

Add missing static keywords to col(1)

Add missing static keywords to cmp(1)

Add missing static keywords to catman(1)

Add missing static keywords to cap_mkdb(1)

Add missing static keywords to c99(1)

Add missing static keywords to banner(1)

Add KLD_DEBUG option.

- Fix potential double mbuf free: M_PREPEND may free mbuf chain and return
NULL but item will still have the reference ot the mbuf chain and will free
it upon destruction.
- Fix memory leak (unfree'd item on error path).

Fix potential double mbuf free: M_PREPEND may free mbuf chain and return
NULL but item will still have the reference ot the mbuf chain and will free
it upon destruction.