vidcontrol(8): Avoid -Wformat-truncation warning.

The vinfo.font_size is unsigned short, so use %hu instead of %d.

config(8): Allow for appending path elements.

Mainly to suppress -Wformat-truncation warning.

crunchgen(1): Allow for appending extensions.

Mainly to suppress -Wformat-truncation warning.

newsyslog - implement 'p' flag

Implement the 'p' flag for newsyslog from NetBSD. This flag results in
the first log file for a given file to not be compressed.

While here, don't change file attributes during a no-op run

Obtained-from: FreeBSD (r327451)

newsyslog - Fix misc issues

When building the command to execute for compression, newsyslog was modifying
the generic arguments array instead of its own copy.
Meaning on the second file to compress with the same arguments, the command line
was not the one expected.
Fix it by creating one copy of the arguments per execution and modifying that
copy.

While here, print the command line executed in verbose mode.

newsyslog - Fix typo

Obtained-from: FreeBSD (r326622)

newsyslog - Flexible compression program execution

Allow newsyslog to execute compression commands which
have a semantic different than the traditional gzip(1)

This is done to allow to use zstd(1) as a compression tool without
having to patch it to change its default behavior.

Obtained-from: FreeBSD (r326617)

newsyslog - Allow writing RFC5424 compliant rotation message.

This modification adds the capability to newsyslog to write the
rotation message in a format that is compliant with RFC5424. This
capability is enabled on a per-log file basis through a new value
("T") in the flags field in newsyslog.conf. This is useful on systems
that use the RFC5424 format for log files so that the rotation message
format matches that of the other log messages. There has been recent
mention of adding an RFC5424 compliant mode to syslogd and at least
one alternative system log daemon (rsyslogd) that already has the
capability to use that format.

Obtained-from: FreeBSD (r318960)

newsyslog - Add a new Y flag to newsyslog.conf

This makes newsyslog use zstandard to compress log files.

Given Z is already taken for gzip and zstandard compression level stands in
between gzip and xz (which has the X flag) chosing Y sounds ok :)

Obtained-from: FreeBSD (r316981)

newsyslog - Fix typo

Obtained-from: FreeBSD (r308457)

newsyslog - More basename/dirname changes

Properly patch up dirname()/basename() calls to not clobber ent->log.

It turns out that we had a couple of more calls to dirname()/basename()
in newsyslog(8) that assume the input isn't clobbered. This is bad,
because it apparently breaks log rotation now that the new dirname()
implementation has been merged.

Fix this by first copying the input and then calling
dirname()/basename(). While there, improve the naming of variables in
this function a bit.

Obtained-from: FreeBSD (r305651)

newsyslog - Clean up basename/dirname calls

Pull copies of the input pathname string before calling basename() and
dirname() to make this comply to POSIX. Free these copies at the end of
this function. While there, remove the duplication of the 's' ->
'logfname' string. There is no need for this.

Obtained-from: FreeBSD (r303449)

newsyslog - Remove unneeded sleep

Eliminate unnecessary sleep(10) when -R and -s are specified

After going through the signal work list, during which do_sigwork()
is called and essentially does nothing because -s and -R were
specified on the command line, newsyslog will sleep for 10 seconds
as the (verbose) code says: "Pause 10 seconds to allow daemon(s)
to close log file(s)".

However, the man page verbiage for -R (and -s) seems quite clear
that this sleep() is unnecessary because the daemon was expected
to have already closed the log file before calling newsyslog.

Obtained-from: FreeBSD (r301532)

newsyslog - treat 'c' flag in the config as 'C'

When -C was introduced in r114137 the plan was to have -C and -c being used for
"create" due to a typo in FreeBSD <= 4.8 a temporary compatibility hack has been
added to make -c being like -G aka GLOB and a warning was issued for the user to
be aware of the futur change for -c.

12 years later it is more than time to remove that hack and finish the what was
intent in r114137

Obtained-from: FreeBSD (r290226)

newsyslog - Allow signal names for config

Obtained-from: FreeBSD (r289879)

newsyslog - Fix typo

Obtained-from: FreeBSD (r289677)

newsyslog - Enhance error mesage

If we fail to send a signal after rotation, print
the pidfile from which the corresponding PID was
obtained.

Obtained-from: FreeBSD (r272763)

newsyslog - Initialize the struct tm

Obtained-from: FreeBSD (r257600)

newsyslog - Fix -Wunsequenced

Obtained-from: FreeBSD (r252378)

newsyslog - Specify the correct filename

We want to stat the archived log file rather than the logfile itself.

Obtained-from: FreeBSD (r251240)

newsyslog - Fix for NFS.

Some filesystems (NFS in particular) do not fill out
the d_type field when returning directory entries through
readdir(3). In this case we need to obtain the file type
ourselves; otherwise newsyslog -t will not be able to
find archived log files and will fail to both delete old
log files and to do interval-based rotations properly.

Obtained-from: FreeBSD (r250545)

newsyslog - Fix interval-based rotations when the -t flag is used

In this case, find the most-recently archived logfile
and use its mtime to determine whether or not to rotate,
as in the non-timestamped case.

Previously we would just try to use the mtime of <logfile>.0,
which always results in a rotation since it generally doesn't
exist in the -t case.

Obtained-from: FreeBSD (r248776)

newsyslog - Rename field.

Rename the run_cmd field to sw_runcmd to make it consistent with the
other fields in struct sigwork_entry.

Obtained-from: FreeBSD (r245963)

newsyslog - Print correct message.

Ensure that newsyslog -n prints the correct message for a rotation rule
that uses the 'R' flag.

Obtained-from: FreeBSD (r245962)

newsyslog - Initialize some fields

When the 'R' flag is used with a newsyslog.conf entry, some fields of
the corresponding struct sigwork_entry were left uninitialized,
potentially causing an early return from do_sigwork(). Ensure that these
fields are initialized, and handle the 'R' flag properly in
do_sigwork().

Obtained-from: FreeBSD (r245961)

newsyslog - Update mtime after archiving.

Make sure to update the mtime of a logfile after archiving it. This
ensures that the next rotation happens at the correct time when using
interval-based rotations.

Obtained-from: FreeBSD (r244997)

newsyslog - Dry-run option implies no root option.

Have -n imply -r, since dry-run mode obviously doesn't require root
privileges.

Obtained-from: FreeBSD (r244996)

newsyslog - Add some missing 'static' keywords

Obtained-from: FreeBSD (r241777)

newsyslog - Signal result checking unneeded

We don't need to check the result of sending signal when -R option is
specified.

Obtained-from: FreeBSD (r238281)

newsyslog - Oversize checks.

Don't run through time checks when entry is definitely oversized. This
leads to newsyslog rotating on (size OR time) if both are specified.

Obtained-from: FreeBSD (r233257)

newsyslog - Add new modifier - "R"

When it is specified the path to pid file will be considered as a
path to a binary or a shell script to be executed after rotation
has been completed instead of sending signal to the process
id in that file.

Obtained-from: FreeBSD (r221873)

newsyslog - Fix an old bug in newsyslog

We kept one log file more than was requested in newsyslog.conf.
This was only the case using the non-time based filenames (.0, .1,
.2 etc.).

The change also makes newsyslog clean clean up the old extra logfile
so users don't end up with a single stale logfile which won't be
rotated out.

This change also cleans up some code a bit to avoid more copy / paste
code and removes some old copy / paste code in the process.

Obtained-from: FreeBSD (r220926)

newsyslog - Sync manpage's SYNOPSIS with program's usage

Obtained-from: FreeBSD (219434)

newsyslog - Make code more friendly to the non-C99 compilers

Obtained-from: FreeBSD (r218944)

catman(1): Suppress few warnings.

bsd.sys.mk: Implement WARNS_AUDIT option.

The buildworld is expected not to pass this mode, so make(1) should
be invoked with '-k'. Strictly just as convenience for developers.
This is similar to FORMAT_AUDIT, for extra diagnostics by compiler at a
given WARNS level. The -Wall and -Wextra are getting bit too crowded.

While there, document FORMAT_AUDIT too.

In-discussion-with: swildner

Remove an unused variable in various Makefile.inc's.

games: Add FALLTHROUGH where missing.

tetris(6): Fix -Wformat-truncation warning.

Also make use of macro instead of relying on magic numbers for keys.

While there, do a similar thing in sail(6).

rogue(6): General cleanup.

 * Move externs to common rogue.h
 * Constify
 * Various cleanups
 * Several bugfixes from NetBSD

phantasia(6): General cleanup.

  * Staticize
  * Constify
  * Collect externs
  * Add attributes
  * Sync logic with NetBSD (except for style)

hack(6): Suppress warnings.

Add FALLTHROUGH directives were needed, fix few format issues.

syscons - Untangle device attachement from isa?, attach to nexus? instead.

* The syscons device was still attaching via the isa? bus for mostly
  historical reasons. Instead directly attach to nexus? and use a custom
  _identify method to create the sc%d child devices.

led(4): Turn this into a module. Only used by acpi_thinkpad and acpi_asus.

games: Do not override default CFLAGS.

Preserve system default CFLAGS. Override is only needed in case of boot2.

libfetch: Suppress fallthrough warning.

Prefer https over http in commit template

Shifting into or out of an int's sign bit is undefined, and this loop
deterministically and intentionally does both.

Dragonfly-bug: https://bugs.dragonflybsd.org/issues/3112
Submitted-by: "mmcc"

bus_dma.9: Add a few lines about bus_dmamap_load_ccb().

While doing, add a comment to the function.

<sys/cdefs.h>: Add __alloc_size2() for functions taking number and size.

Add it to calloc().

Pointed-out-by: zrj

<sys/cdefs.h>: Make __nonnull variadic.

Make use of it in the remaining __nonnull decorations we have.

lvm(8): Add line editing support (libedit is already linked).

libedit: Check dereferenced pointer for NULL.

Already fixed in NetBSD.

Caught-by: gcc80 -Wpointer-compare

Fix two more gcc80 -Wsizeof-pointer-memaccess warnings.

pppctl(8): Avoid overlapping strcpy() copies.

Use intermediate.

telnet(1): Avoid overlapping strncpy() copies.

The cmdrc() is shifting string to the left, use temporary.

While there, do some constification.

pax(1): Honour the restrict in sigaction().

Use a setup_sig() helper and make it fail when either of sigaction fails.

While there, do not leak fds for "." + minor cleanup.

Taken-from: OpenBSD

dd(1): Use a local swapbytes() function.

The swab(3) has restrict qualifiers for src and dst.
Avoid relying on undefined overlapping swab behavior.

Taken-from: OpenBSD

mille(6): General cleanup.

  * Constification.
  * Braces.
  * Do not use increments/decrements on booleans.
  * Add few missing fallthrough.

pf: Fix bzero() argument.

Intention here is to zero out the local pfioc_trans structure and not the
char *t puinter.

Caught-by: gcc80 -Wstringop-overflow

ktrdump(8): Fix bzero() argument on error.

Intention here is to zero out the struct and not the local pointer,
based on change in e7c0dbbaa.

Caught-by: gcc80 -Wstringop-overflow

vinum(8): Allow for terminating NULL.

Mainly to suppress -Wstringop-overflow warning.

cvs(1): Avoid -Wstringop-overflow warning.

The cvs will terminate after calling it's error(1, ...) but it can not be
marked as noreturn. So just add a hint for compiler that memcmp() with
read1 == -1 as an argument is not reachable.

Fix gcc80 -Wbool-operation warnings in fortune(6) and hack(6).

Fix a number of gcc80 -Wsizeof-pointer-memaccess warnings.

The file is reused in various utilities.

However, the mid- to long-term plan is to remove legacy utmp code and
only go with standard utmpx.

gnu/lib: Raise WARNS generally to 1 via Makefile.inc.

libm: Avoid left-shifting negative integers.

It is undefined behaviour.

Taken-from: OpenBSD

ipfw: Avoid left-shifting negative integers.

It is undefined behaviour.

While there, mark one fallthrough case.

games: Fix -Wint-in-bool-context warnings.

These cases were the last two in base.

binutils227: Fix comparison of pointer against char.

Backport from upstream.

binutils225: Fix comparison of pointer against char.

Backport from upstream.

While there, deal with -Wshift-overflow in aarch64.cc (unused).

gdb: Fix wrong comparison of pointer against char.

Backport from upstream.

Oops, fix up my previous commit properly.

Fix gcc80 -Wmultistatement-macros.

gcc50/Makefile.inc: Add newline at EOF.

Revert "terminfo: Add xterm-termite to Makefile.entries"

This reverts commit 5f4713d27c03fbf254d1d7e9939f9864261337b6.

gnu/sbin: Raise WARNS generally to 1 via Makefile.inc.

gcc50/libgcov: Fix whitespace in Makefile.

Revert "kernel: Disable attributes that take no effect."

This reverts commit 191430ddf0486f992e75ff14effbb5a69e1da2bd.

Needs more checking.

sbin/mount_autofs: Add description

TL;DR users won't be running this command to mount autofs.

Having autofs_enable=YES won't run this command either.
There is a high level command automount(8) and that does everything
based on /etc/auto_master contents.

mount_autofs(8) is a low level version where users can manually
specify things automount(8) internally passes to mount(2).
Thus it's not easy or needed to understand what -fOp options exactly
do without having clear understanding of autofs internal.
But I'll write it here in case interested.

 -f is a string which eventually gets copied to statfs::f_mntfromname.
  Unlike most of the filesystems with backing store, this is just
  a label to identify the autofs mount.
 -O is mount options of autofs(5).
 -p is a prefix string to distinct direct and indirect mapping.

Fix some -Wunused-const-variable warnings and a -Wmisleading-indentation.

dconschat(8): Put the fflush() under 'verbose' too.

dconschat(8): Fix -Wmisleading-indentation.

Fix one last -Wnonnull-compare in m4(1).

Fix gcc80 -Wnonnull-compare.

For arguments that are checked against NULL in the function, we don't
want a nonnull attribute, because that will cause the NULL check to be
optimized away in higher -O levels and it will also trigger a
-Wnonnull-compare warning.

See also 0f037c78525bfb7df49de8338826e15137859b1d commit message.

lib/i18n_module: Add a missing break.

It is intended and correct, but has no correctness implications due
to is94 => is96.

Confirmed-by: cem@freebsd, eadler@freebsd

rcs: Fix gcc80 warnings (-Wdangling-else and -Wmisleading-indentation).

hammer2: Remove duplicate initialialization of fields.

Avoids -Woverride-init diagnostics.

While there, minor whitespace cleanup.

kernel: Disable few nonnull attributes.

To avoid -Wnonnull-compare, similarly as in:
0f037c78525bfb7df49de8338826e15137859b1d

devfs_core.c:   devfs_find_device_by_name()
1st arg checked (NULL == return)
kern_slaballoc.c: kfree()
1st arg checked (NULL == panic)
2nd arg not checked
kern_synch.c:   tsleep()
1st arg checked (NULL == KASSERT)
subr_prf.c: kvcprintf()
1st arg checked (NULL == "(fmt null)")
subr_sbuf.c: sbuf_vprintf()
2nd arg checked (NULL == KASSERT)

kernel/bus: Avoid panic on NULL passed to kfree().

Our kfree() will issue panic() on NULL pointer.

Caught-by: gcc80 -Wnonnull

kernel/ipsec: Avoid -Warray-bounds warning.

Just panic if something else than 0, 1 or 2 is passed.

kernel/ipsec: Fix useless m_pkthdr.len check.

Compare it against sadb_msg.

Caught-by: gcc80 -Wtautological-compare

ig_hal: Reindent e1000_ich8lan.c a bit.

No functional change.

ipfw3_nat: Reindent switch cases.

No functional change.

While there, make add_redir_spool_cfg() and del_redir_spool_cfg() static.

kernel: Disable attributes that take no effect.

Both gcc50 and gcc80 ignore these attributes:
warning: ignoring attribute 'aligned' because it conflicts with attribute 'packed' [-Wattributes]
No differences for final objects.

netgraph: Disable unused structs.

This needs recheck.

While there, document that musycc and if_mn requires netgraph and
are not compatible with netgraph7.

usr.sbin/autofs: Drop unneeded header include

common.c has no dependency on autofs api itself.

Sync zoneinfo database with tzdata2018b from ftp://ftp.iana.org/tz/releases

* Sao Tome and Principe switched from +00 to +01.

* Brazil's DST will now start on November's first Sunday.

* A discrepancy of 4 s in timestamps before 1931 in South Sudan has
    been corrected.

* The abbreviation invented for Bolivia Summer Time (1931-2) is now
    BST instead of BOST.

* Various smaller adjustments.

For a detailed list of changes, see share/zoneinfo/NEWS.

limit(1): fix always true condition

drm/i915: Backport fix for a bad bitmask.

No-op, we explictly disable stolen memory in i915_gem_init_stolen().