If we get back too much data to fit in result, return NULL.  This avoids
a buffer overflow, but might negatively impact those hosts who have
enough aliases to fill MAXHOSTNAMELEN * 2 characters in them.

Good candidate for merging back into -stable.  Lightly tested by me, but
it came from OpenBSD a while ago.

Obtained from: OpenBSD

Use correct size for readlink buffer.  This is the size of the
buffer -1 since readlink adds its own NUL to the end.

Inspired by: Similar changes in OpenBSD

use strncpy correctly
use strncat correctly
check buffer size in graph_name
Change paths from /tmp to /var/run for server sockets

This has been building in my tree for ages, but has remained lightly
tested.
Reviewed by: jkh ages ago

o be careful about buffer overflows
o Use snprintf rather than sprintf

This is only lightly tested.

Remove unused strvisx.

Don't assume that hp->h_lenght == 4.  Be conservative in its use.
Submitted by:  J. Assange a long time ago.

o Use snprintf rather than sprintf.
o Revert gid to caller before a shell so we can run setgid.
o minor style nits to make bruce happy :-)
o use strncpy correctly.

Use mkstemp rather than mktemp for yacc's temp files.  This change was made
to OpenBSD a long time ago and to my tree shortly thereafter.  I think theo
made this change, or one similar to it, but I could be wrong.

o Use snprintf rather than sprintf
o Add more checks for buffer overflows
o Use snprintf rather than strcat/cpy and have better checks for max
  length exceeded.

Most of these changes are not exploitable buffer overruns, but it never
hurts to be safe.

Inspired by and obtained from: OpenBSD

o Use snprintf over sprintf.
o Use strncpy correctly.
o Use enough buffer for line.

Inspired by or Obtained from: Similar changes in OpenBSD

Make sure we pass the length - 1 to readlink, since it adds its own
NUL at the end of the path.

Inspired by: OpenBSD's changes in this area by theo de raadt

$@ is deprecated, use ${.TARGET} instead.  Ditto for other single character
$n.
PR:  conf/3273

Make sure we don't overflow the path buffer.  Exit if we do.
Obtained from or inspired by: A similar change in OpenBSD by theo

Make sure we pass the length - 1 to readlink, since it adds its own
NUL at the end of the path.
Inspired by: OpenBSD's changes in this area by theo de raadt

Updated generated files.

Don't generate COMPAT_43 cruft if there are no COMPAT_43 syscalls.
In particular, don't generate an include of "opt_compat.h" if it
wouldn't affect anything we create.  This will fix recent breakage
of the ibcs2 LKM.  The ibcs2 syscall files were not regenerated
properly, so the LKM didn't break immediately when we started
generating this extraneous include.

Include "opt_compat.h" explicitly instead of depending on
makesyscalls.sh generating it as pollution.

Don't attempt to separate the includes by newlines.  makesyscalls.sh
just ignores newlines.

Fixed `make depend', which I broke here about a month ago by moving
the null default for `depend' from bsd.dep.mk to bsd.obj.mk.
bsd.kmod.mk included these files in a bad order, so it picked up
the null default.

The version of the softdep changes in FreeBSD broke the
(doingdirectory && !newparent) case of ufs_rename().
rename("D1/X/", "D2/Y/") gives a wrong link count for D2.

Submitted by: Bruce Evans <bde@zeta.org.au>
Reviewed by: Kirk McKusick <mckusick@McKusick.COM>

- Added entry for Xircon CreditCard Modem.

Submitted by:  "Mike Del" <repenting@hotmail.com>

My previous change could conceivably succeed at removing CHROOTDIR.
Ignore errors from the chflags command that follows it, just in case.

Don't let ifunit() modify the string passed as an argument.
it may be in the text segment and write protected.

Correct ``set server'' usage and add mention that ``set ns'' changed
in README.changes.
Suggested by: stuart henderson <stuart@internationalschool.co.uk>

Remove some junk left over from a previous commit.
Submitted by: phk

Print the write counts if they are nonzero even if we're mounted
readonly, since they tell us about previous write activity.

Use the correct format to print the write counts.

Updated generated files.

Fixed some style bugs in output (missing tabs and unparenthesized macros).

Fixed some style bugs in source (mostly, superfluous backslashes).

Add LIB_SPEC (same as on i386) so that gcc knows about libc_r.

Fixed pedantic semantics errors (bitfields not of type int, signed int
or unsigned int (this doesn't change the struct layout, size or
alignment in any of the files changed in this commit, at least for
gcc on i386's.  Using bitfields of type u_char may affect size and
alignment but not packing)).

Fix more of my DDB breakage.

Fix a typo which prevented i386 elf from working at all (including Linux
emulated elf binaries).

Correct even more ioctl breakage :(
Doesn't people check their code before committing anymore, it could
at least compile  ????

Sync with sys/i386/isa/clock.c revision 1.123.

Merge 64bit portability fixes from sys/i386 stuff.

Fixed pedantic semantics errors (bitfields not of type int, signed int
or unsigned int (don't do this in one case where it would change the
object file)).

Make DDB work again after I broke it :-(.

Add rcsid, remove unused #includes. Use err(3).

Fix spelling in printf().

Use warnx()+fprintf() to handle multi-line messages. Correct K&R support and
KNF continuation indent rule.
Requested by: Bruce.

Updated yet another ioctl, and put wst in LINT to inhibit further bitrot.

Spelling fixes.

Submitted by: Josh Gilliam <josh@quick.net>
PR: 6880

ip_fil.h has 9 separate declarations of iplioctl() in a disgusting
ifdef tangle.  The previous commit to ip_fil.h didn't change the
one that actually applies to the current FreeBSD kernel, of course.
Fixed.

Fixed style bugs in previous commit to ip_fil.h.

Reword to remove confusion between su(1)'s "-c" option and that of the shell
being invoked.  Add example that clarifies usage of "-c".

PR: 6859

Disable attempted write combining support.  This probably
causes more trouble than help (for now.)

Fixed an ioctl which grew overnight.

Backed out previous commit, which just consisted of a 1-byte editing
error.

Fixed missing ifdefs and disordering in previous commit.

Fixed ifdef bogotification in previous commit.

Fix typo: exit -> exist

Speed up the cleaning of CHROOTDIR by almost a factor of two, by
replacing two monstrous file tree walks with one monstrous tree walk
and two miniscule ones.

Add a member function more to the timecounters, this one is for use
with latch based PPS implementations.  The client that uses it will
be committed after more testing.

Fix another DEVFS/SLICE ioctl.

This is a prototype implementation of the draft-mogul-pps-api-##.txt
paper.

It will be updated along with the draft and possible subsequent
standard.

The ppbus based pps driver is updated to implement this API.

Fix some more ioctls which I missed becausese they were hidden by options
which were not in LINT.

Fix the retyping of the ioctl cmd parm from int to u_long.

Always ask for homedir.

PR: 6754
Submitted by: uhlar@netlab.sk

This commit fixes various 64bit portability problems required for
FreeBSD/alpha.  The most significant item is to change the command
argument to ioctl functions from int to u_long.  This change brings us
inline with various other BSD versions.  Driver writers may like to
use (__FreeBSD_version == 300003) to detect this change.

The prototype FreeBSD/alpha machdep will follow in a couple of days
time.

Fix previous commit.  bit 1 is the execute bit, not the read bit *blush*
ie, BINMODE=4551 -> BINMODE=4554
Pointed out by: Bruce Evans <bde@zeta.org.au>

PR: docs/3636
Submitted by: Gary Palmer gpalmer@FreeBSD.ORG
Add mention of the 1024-character line length limit on the netgroup database.

Sigh, all good words are reserved words these days...
s/private/priv/

Noticed by: sos

Added a used include (in ifdefed code).

Fixed pedantic semantics errors (bitfields not of type int, signed int
or unsigned int).

Fixed pedantic syntax errors caused by a trailing semicolon in a macro
definition.

Null change.  Forgot to mention in previous log message that MNT_NOATIME
is now ignored for special files, so that mounting root with option
noatime doesn't break reporting of idle times in programs like `w'.
The problem of execessive disk updates just to stamp atimes will be
handled for special files by only writing atimes to disk when inodes
become active.  This works well because special files are relatively
uncommon and their atimes are even more disposable at panic time than
regular files' atimes.

Compile mtree before it is used, this is a hack, but make world has
failed since new functionality was introduced in mtree.

Fixed some longstanding timestamp bugs:
1. mark atimes and mtimes of special files and fifos for update upon
   successful completion of non-null i/o, not at the beginning of the
   syscall.
2. never update file times for readonly filesystems.  They were updated
   for stats and closes but not for syncs.  The updates were of course
   only in-core and were thrown away when the inode was uncached, so
   the times sometimes appeared to go backwards.

Improved comments in code related to (1) (mostly by removing them).

Unmacroized ITIMES().  The test in (2) bloated it even more.  Don't
call getmicrotime() in the function version of it when we only need
the time in seconds.

Sync with sys/i386/isa/clock.c revision 1.122.

Add a "this" style argument and a "void *private" so timecounters can
figure out which instance to wount with.

Use a relative libalias.so name (hardcoded at 2.5 - yeuch!).

Search for libraries in dlopen() when the specified path
contains no ``/''s.
Elf already searches it seems.
Mostly submitted by: Mike Smith <mike@smith.net.au>

Make ppp world-readable so that root can get at it over
NFS when nobody isn't in group network :-)

Pointed out by: Bruce Evans <bde@zeta.org.au>

Update the write descriptor selection set for our data links
*after* shuffling fragments from the IP queue into the
individual link queues.

This fixes a latency problem pointed out by: Bruce Evans <bde@zeta.org.au>

Fix an alignment problem on alpha by doing a bytewise copy.

Make ``set ? log'' more verbose.
Suggested by: Paul Dufresne <dufrp@oricom.ca>

Don't call PunchFWHole() ifdef NO_FW_PUNCH
Pointed out by: "Steve Sims" <SimsS@IBM.Net>

Make sure the default value of a dummy variable is 0
so that it doesn't do anything.

Put the correct pid in /var/run/tunX.pid
Suggested by: many

Fix wrong data type for a pointer.

clean up the changes made to ipfw over the last weeks
(should make the ipfw lkm work again)

Fix this so that it compiles in the !__STDC__ case.
Also be consistent about usage of #if ...

Pointed out by: bde

keymap -> key_map so that the kernel will compile with
-DESKEYMAP.

PR: 6864
Submitted by: Javier Rueda <jmrueda@diatel.upm.es>

PR: docs/5986
Submitted by: MITSUNAGA Noriaki mitchy@er.ams.eng.osaka-u.ac.jp
Replace use of obsolete .q groff macro with .Dq

PR: docs/5628
Submitted by: MITSUNAGA Noriaki mitchy@er.ams.eng.osaka-u.ac.jp
Correct formatting error in display of the "-i size" option description section.

Sync PATH variable with 'dot.login'.

I think here is not my area so feel free to back this out when any
troubles are happened.

Pointed out by: IIJIMA Akihiro <aki@noc.titech.ac.jp>

Add a warning message for a thread locking against itself. This is
not supposed to happen, but I have seen bogus g++ code that causes
it.

Simplify the handling of thread specific data. Only track if a key
is allocated or not, rather than keeping a count and attempting to
know it it is in-use. POSIX says that once a key is deleted, using the
key again results in undefined behaviour.

Re-design the thread specific key structure.

named-xfer can't be linked static due to the duplicated symbols in
libc and libbind.

rpc.rstatd required libkvm.

Only try to build these on i386 for the time being.

I got the last commit back to front.

Spelling corrections.

PR: 6868
Submitted by: Josh Gilliam <josh@quick.net>

Make LINE30 a new style option.

Make BS_TARG_SAFEMODE a new style option.

Spelling nits.

Pointed out by: Sue Blake <sue@welearn.com.au>

-DALLLANG is now obsolete in "src/Makefile"; move to "src/release/Makefile".

PR: 6685

Don't attempt to copy the whole slices "struct" for DIOCGSLICEINFO.
The slices "struct" isn't really a struct; we allocate only part of
it in the fully dangerously dedicated case.  Since the "struct" is
malloced, the page beyond it may not be mapped, so attempts to copy
it would crash.  This problem became larger when the full struct was
bloated from < 1K to > 3K by the addition of (mostly unused) DEVFS
tokens some time before 2.2.0 was released.

Removed dead code.

Don't run the target's ldconfig or build it as a tool.  If target == host
(in particular, if DESTDIR is empty or "/"), then the host's ldconfig will
be the target's ldconfig by the time it is run.

Fixed disordering of env.

Don't know too much about libcrypt.  Use exactly the same definition of
_libcrypt as lib/Makefile.

Don't build strip twice.

Fixed bogotification of the lex bootstrap in rev.1.186 and the previous
commit - don't wander off to bootstrap mtree and include in the middle
of bootstrapping lex, and don't forget what we were doing and build
some lex obj dirs twice.

Reviewed by: Kirk Mckusick (mckusick@mckusick.com)
Submitted by: luoqi Chen
fix a type in fsck.
(also add a comment that got picked up by mistake but is worth adding)

Fix the signal behaviour for internal states which set the thread
state to running despite the SA_RESTART flag which is really just for
syscalls.