Use asprintf instead of malloc + snprintf and test the memory allocation

Remove dead declaration

PR: 195928
MFC after: 2 days

Export a list of VM objects in the system via a sysctl.  The list can be
examined via 'vmstat -o'.  It can be used to determine which files are
using physical pages of memory and how much each is using.

Differential Revision: https://reviews.freebsd.org/D2277
Reviewed by: alc, kib
MFC after: 2 weeks
Sponsored by: Norse Corp, Inc. (forward porting to HEAD/10)

Merge sync of head

Merge from head

Remove extendline definition
Remove now unused PWBUFSZ define

Use calloc(3) instead of malloc(3) + memset(3)
While here check the return of calloc(3)

The initial logic for allocating the new string was wrong, the conversion
to strndup(3) duplicated the same mistake, actually strdup(3) is good enough
to allocate the new string.

Use strndup(3) instead of malloc(3) + memcpy(3)
Check the return of strndup

Remove useless call to extendarray

The pwdb function is only used once to check the database rename it
pwdb_check and simplify it accordingly

if the check of the pw db fails return the failed value

Some style(9) fixes

Replace sprintf(3) with snprintf(3)

Use snprintf(3) instead of sprintf(3)
Remove useless "else"

Remove now unused LNBUFSZ buffer size

Return from the function as early as possible

This reduces the depth of the if statements and improves clarity of the code

Remove some uneeded headers

Remove unneeded headers

Replace malloc(3) + strcpy(3) + strcat(3) by asprintf(3)

Use snprintf(3) instead of strcpy(3) + strncat(3)

Use sbuf(9) instead of homebrewed buffered string

Partially revert 278642

On reinstall (overwrite) install(1) does not handle chflags

Reported by: ian

Use PRECIOUSPROG instead of custom code to handle schg

This allows to preserve schg when installed with -DNO_ROOT

MFC after: 1 week

Clean up more usb related files when MK_USB == no when dealing with
manpages, libraries, and binaries

MFC after: 1 week
X-MFC with: r278135
Sponsored by: EMC / Isilon Storage Division

Revert r277652

uid and gid are never and should never be negative. The pw(8) manpage clearly
states the -u and -g arguments are for uids/gids, hence using negative values is
abusing a bug in former versions of pw(8)

gr_equal(): Fix a crash that could occur if the first group's member list
was longer than the second's. There is no need to compute and compare the
member list lengths in a separate pass, since we now just return false when
comparing member names if the list lengths are not equal.

MFC after: 2 weeks

Allow negative numbers in -u and -g options

PR: 196514
MFC after: 1 week

Convert to usr.bin/ to LIBADD
Reduce overlinking

Convert to LIBADD.

Reviewed by: bapt

Sync to HEAD@r274766

Merge from head@274682

Fix wrong message when using pw -V with a non existent directory
Add a regression test about it

PR: 194971
Submitted by: Freddy DISSAUX <bugzilla@dsx.bsdsx.fr>

Sync to HEAD@r274095.

Fix renaming a group via the gr_copy function

Add a regression test to pw(8) because the bug was discovered via using:
pw groupmod

PR: 187189
Reported by: mcdouga9@egr.msu.edu
Tested by: mcdouga9@egr.msu.edu
Patch by: Marc de la Gueronniere

Ensure pw userdel -u <invalid> do not try to remove root

Check the uid passed is actually a number as early as possible

MFC after: 1 week

Do not delete the group wheel when bad argument is passed to pw groupdel -g

Check that the -g argument is actually a number, if not report an error.
This argument is converted without checking with atoi(3) later so without this
check it converts any alpha entries into 0 meaning it deletes the group wheel

Add a regression test about it

PR: 90114
Reported by: bkoenig@cs.tu-berlin.de
MFC after: 1 week

Fix a regression in pw usermod -G list

The user was perperly adding the to different groups from "list" but was not
removed from the other groups it could have belong to.
While here add a regression test about this bug

PR: 185666
Reported by: sub.mesa@gmail.com
MFC after: 1 week

When a group is renamed then the group has been invalidated for sure.
In that case get the group information using the new name.

Add a regression test about this bug

PR: 193704
Reported by: az

Merge HEAD@r272944.

Two more places where login_setcryptfmt() defaults to MD5 were missed
in r252688.

MFC after: 3 days

- Add a test for bug 191427 where pw(8) will go into an infinite loop

Reviewed by: will
MFC after: 1 month

Merge head from 7/28

Convert devd's client socket to type SOCK_SEQPACKET.

This change consists of two merges from projects/zfsd/head along with the
addition of an ATF test case for the new functionality.

sbin/devd/tests/Makefile
sbin/devd/tests/client_test.c
Add ATF test cases for reading events from both devd socket types.

r266519:
sbin/devd/devd.8
sbin/devd/devd.cc
Create a new socket, of type SOCK_SEQPACKET, for communicating with
clients. SOCK_SEQPACKET sockets preserve record boundaries,
simplying code in the client. The old SOCK_STREAM socket is retained
for backwards-compatibility with existing clients.

r269993:
sbin/devd/devd.8
Fix grammar bug.

CR: https://reviews.freebsd.org/rS266519
MFC after: 5 days
Sponsored by: Spectra Logic

Minor style tweaks.

Obtained from: OpenBSD (CVS rev. 1.7)
MFC after: 3 days

pw: fix up deletion of users from groups

Previuosly given 'foo,bar' members, removing 'foo' would result in an
infinite loop.

PR: 191427
Submitted by: Voradesh Yenbut <yenbut cs.washington.edu>
MFC after: 1 week

Fix changing the username

PR: 189172
Submitted by: fullermd@over-yonder.net
MFC after: 1 week

Removed compatibility with pre FreeBSD 2.2 pw_mkdb command
Fix some broken indentattion

PR: 189173
Submitted by: fullermd@over-yonder.net
MFC after: 1 week

use .Mt to mark up email addresses consistently (part2)

PR: 191174
Submitted by: Franco Fichtner  <franco@lastsummer.de>

IFC @266473

Merge from head

Use src.opts.mk in preference to bsd.own.mk except where we need stuff
from the latter.

Merge head

Merge head

IFC @264767

Simplify reading pw.conf(5) by using getline(3)

Fix pw(8) deletion of group "username" on userdel even if group "username"
is not associated with user "username". E.g., user "foo" has primary group
"wheel" and is unassociated with group "foo", yet userdel would delete the
group "foo" when deleting user "foo" (despite the fact that user "foo" is
not associated with group "foo" in any way).

Patch committed with minor style(9) changes.

PR: bin/169471
Submitted by: Alexander Pyhalov <apyhalov@gmail.com>

Part 2 of bug 187310.. had to commit separately due to local confusion.
Don't let pw crash when give certain input.

PR: 187310
Submitted by: Kim Shrier
MFC after: 1 week

Stop pw(8) from segfaulting when given certain input

PR:187310
Submitted by: Kim Shrier
Obtained from: bug
MFC after: 1 week

mdoc: minor paragraph fixes.

IFC @256277

Approved by: ken (mentor)

Merge from head

libutil: Use O_CLOEXEC for internal file descriptors from open().

IFC @253398

Fall back to sha512 if passwd_format is not set.

MFC after: 3 days

There is no reason to disallow setting the password or account expiry
date to the current date.

MFC after: 3 days

Fix -Wunsequenced warning

Submitted by:   dt71@gmx.com

mdoc sweep

sync from head

commit correct tested fix for gr_util.c

Approved by: theraven

Cleanup gr_add() so it does not leak mem
This is part of ongoing work on sbin/pw

M    libutil.h
M    gr_util.c

Approved by: theraven

IFC @247348.

Sync with HEAD.

Correct the r242545 sync with head@242525.

Prevent a null pointer dereference in pw userdel when deleting
a user whose group != username.

IFC @ r245509

Remove unused variables

Approved by: cperciva
MFC After: 3 days

libutil: fix typo in comment for gr_fini.

Submitted by: Christoph Mallon <christoph.mallon gmx.de>

libutil: utilize strsep instead of strcat in a loop in gr_make

Submitted by: Christoph Mallon <christoph.mallon gmx.de>

libutil: move group_line_format into the scop of its only user.

Submitted by: Christoph Mallon <christoph.mallon gmx.de>

libutil: eliminate 'found' variable in gr_equal

Submitted by: Christoph Mallon <christoph.mallon gmx.de>

IFC @ r245178

pw: free group returned by gr_add

IFC @ r244983.

Simplify pointing dst after the end of all the gr_mem pointers in newgr

Submitted by: pjd
Reviewed by: db

errno = ENOMEM was supposed to be removed not return (NULL);

Submitted by: gcooper

malloc() sets errno to ENOMEM already.

Submitted by: Christoph Mallon <christoph.mallon@gmx.de>

Do not leave parts of the new group uninitialized in gr_dup().

Submitted by: Christoph Mallon <christoph.mallon@gmx.de>
Reported by: pjd

avoid arithmetic on uintptr_t

Submitted by: pjd
Reviewed by: jilles

cast to uintptr_t to properly calculate offset

Reported by: mdf
Submitted by: db

Add O_CLOEXEC to flopen

Requested by: jilles

Simplify vnextgrent and vnextpwent reusing pw_scan and gr_scan from libutil.

gr_dup: simplify duplication of group

Submitted by: db

- Clean up previous gr_add use malloc instead of calloc
- Fix tinderbox error

Submitted by: db

Fix a regression in "pw group show" introduced r242349:
print a newline after printing each group line.

PR: bin/174731
Submitted by: Jan Beich <jbeich@tormail.org>

Simplify the code by using the new gr_add function

New gr_add function to provide a clean and safe method to append a new member
into an existing group.

Submitted by: db

Use flopen(3) instead of open(2) + flock(2)

Simplify copying of group members by using memcpy

Submitted by: Christoph Mallon <christoph.mallon@gmx.de>

Fix off-by-one error in memory allocation: j entries, one new and a null
terminator is j + 2.

Submitted by: Christoph Mallon <christoph.mallon@gmx.de>