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

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>

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

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

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

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

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)

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

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.

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>

In case of the deletion of a user those whole database has to be regenerated,
otherwise the user planned to be deleted remain in the pwd.db while removed from
the plain text password file.

Fix creating a user and adding it to a group

Reported by: "Sam Fourman Jr." <sfourman@gmail.com>, dim

Simplify string duplication: use strdup instead of malloc + strcpy

Submitted by: db
Approved by: cperciva
MFC after: 2 weeks

Avoid overflowing the file buffer

Submitted by: db
Approved by: cperciva
MFC after: 2 weeks

Use strdup instead of malloc + strcpy

Submitted by: db
Approved by: cperciva
MFC after: 2 weeks

Avoid overflow of file buffer

Submitted by: db
Approved by: cperciva
MFC after: 2 weeks

Remove useless check for NULL prior to free.

Approved by: cperciva
MFC after: 2 weeks

IFC @r243836

MFC

MFC @ r241285

In NIS mode first chmod(2) the temporary file  and is succeed then rename(2)

only rename(2) after chmod(2) has succeed
report error if chmod(2) fails

Reported by: jh

Correctly set the password file mode after renaming in NIS mode

change mode the group file to 0644 after a successfull rename(2)

IFC @ r243164

Avoid possible null deref if ypclnt_new returns null

PR: bin/172979
Submitted by: Erik Cederstrand <erik@cederstrand.dk>
Approved by: cperciva
MFC after: 3 days

IFC @ r242684

Teach pw(8) about how to use pw/gr API to reduce code duplication

MFC after: 2 months

backout r242319, racy and not done in the right place

Reported by: Garrett Cooper  <yanegomi@gmail.com>

make pw_init and gr_init fail if the specified master password or group file is
a directory.

MFC after: 1 month

Do not treat empty name as an uid 0

Reported by: Robert Bonomi <bonomi@mail.r-bonomi.com>

Remove useless atoi(3), previous strspn(3) makes sure that a_name->val is a
number.
This also allow pw user show to work as expected.

PR: bin/172112
Submitted by: "Ilya A. Arkhipov" <rum1cro@yandex.ru>
MFC after: 1 month

Make sure that each va_start has one and only one matching va_end,
especially in error cases.

Remove a reference to CVS and to freefall from a user-facing man page.

PR: docs/171658
Reported by: Chris Petrik (c dot petrik dot sosa at gmail dot com)
Approved by: gabor (mentor)
MFC after: 5 days

IFC @238412.

IFC @ r238370

Revert user comparison back to user names as some user can share uids (root/toor
for example)

get the username information from old_pw structures to still allow renaming of a
user.

Reported by: Claude Buisson <clbuisson@orange.fr>
Approved by: des (mentor)
MFC after: 3 weeks

mdoc: add missing width argument to Bl -tag.

MFC

A new jail(8) with a configuration file, ultimately to replace the work
currently done by /etc/rc.d/jail.

MFC after: 3 months

IFC @ r234692

sys/amd64/include/cpufunc.h
sys/amd64/include/fpu.h
sys/amd64/amd64/fpu.c
sys/amd64/vmm/vmm.c

 - Add API to allow vmm FPU state init/save/restore.

FP stuff discussed with: kib

Remove trailing whitespace per mdoc lint warning

Disussed with: gavin
No objection from: doc
Approved by: joel
MFC after: 3 days

When using uidstart in /etc/adduser.conf, get the next
available user id and show it in the "Uid [xxx]" prompt.

PR: 163863
Submitted by: Moritz Wilhelmy (mw at wzff dot de)
MFC after: 2 weeks

Handle NULL return from crypt(3). Mostly from DragonFly

readlink len-1

Remove unnecessary cast

Detect file modification properly by using tv_nsec.

POSIX 2008 standardizes st_mtim, meaning we can simply use nanosecond
precision to detect file modification.

MFC after: 2 weeks