Remove unused includes. Make it WARNS=6 friendly. Concerning bin/2442, make
a new function dup_shell() to replace ok_shell() and make it unconditionnally
strdup() its result to make the caller's code simplier. Change ok_shell() to
just return an integer value suitable for tests (it was used mainly for that
purpose). Do not use strdup() in the caller's code but rely on dup_shell()
that will do the job for us.

PR: bin/2442

add missing endusershell() call. Original version was incorrect.
PR: bin/2442
Reviewed by: Friedemann Becker <zxmxy33@mail.uni-tuebingen.de>

Add a `-H <fd>' option that is like `-h <fd>', but accepts an already
encrypted password on the specified file descriptor.

PR: bin/22033
MFC after: 2 weeks

(mostly) Clean up some const warnings here.  The code takes some liberties
because it is the originator of various const strings and knows that they
came from malloc.

ANSIfy, WARNSify, CONSTify. Bit of style(9)-ify.

mdoc(7): Properly mark C headers.

mdoc(7): Use the new feature of the .In macro.

Do not compare unsigned int values with ULONG_MAX. The comparison is
always false on 64bit platforms and GCC 3.3.1 issues warning there.

Typo.
This has worked so far because the variable was empty by default.

Submitted by: Kostyuk Oleg <cub@cub.org.ua>

s/warn/info/

Add support for the special shell nologin.

Prodded by: mikeh

Update my email address.

Don't forget the -r on the second password prompt.

PR: bin/53550

Tidy up. Sort headers.

The .Xr utility. Spelling. Put NIS under .Tn

Add information about the log file.

PR: 35575
Submitted by: "Simon L. Nielsen" <simon@nitro.dk> (original version)

Make GCC 3.3 happy with rcsid[], sccs[], and copyright[].

Brucify.

Correctly detect the case where a password entry was changed while we were
preparing to edit it.

PR: bin/50563

Avoid clobbering the gecos field, as we intend to use the original later.

PR: bin/50563

Apply the correct fix for bin/50679: don't mess around with process groups
or the tty, just block selected signals in the parent like system(3) does.
Many thanks to bde for his assistance in finding the correct solution.

PR: bin/50679

Band-aid for the "^C kills the editor" problem.  I haven't yet found the
proper way to fix this.  The way this works is to prepend "exec " to
the editor command to eliminate the "shell in the middle" which prevents
us from properly reawakening the editor after a SIGTSTP.

PR: bin/50679

style.Makefile(5)

Document the -g option in the usage message.

Approved by: markm (mentor)(implicit)

Add a -g option to specify a default login group.

Approved by: markm (mentor)(implicit)

o initialize a couple of local flags so the user can re-edit groups
  next time the subroutine is re-entered
o s/configrun/configflag/
o Make the prompt make sense if the user was creating a configuration file

Approved by: markm (mentor)(implicit)

Make pw_edit() use /bin/sh to interpret the EDITOR environment
variable.

PR: 48748
Reviewed by: mike (mentor)

mdoc(7) police: Scheduled sweep.

Update copyright.

Condense the output to one line per removed user. For user's who
want the more verbose version, there's a -v option.

Approved by: markm (mentor)

There is no sense to use random random() and arc4random() in the same program.
Switch to arc4random() completely.

Revisit the shell special characters issue and settle it once-and-forall.
All characters will be accepted, and pw(8) can sort out which ones it
will allow and which ones it won't.

Approved by: markm (mentor)
Prodded by: Philippe Bourcier <philippe@cyberabuse.org>

Change the behaviour of adduser to match the previous incarnation a little more.

If any of the given groups do not exist complain and let the user try again.
This saves the user from discovering at the end of the process that they've
forgotten to add a group or they've typoed.

Thanks to cmc/dougb for pointing out how bad my sh fu actually is.

Original code by: me
Scary sh rewrite by: dougb

Reviewed by: dougb

Better gender-neutral language.

Submitted by: sheldonh

This is just a forced-commit to note that the previous commit addresses
PR 28773 (not 28733!).  I have closed both PR 28773 and PR 46890.

Changes so the 'pw' command will allow '$' as the last character in a userid
or group name (mainly for the benefit of samba).  This pretty much rewrites
he pw_checkname() routine, but should work exactly the same except for the
above change, and that error messages are somewhat more informative.

PR: 28733 46890
Inspired by: example patch written by Terry Lambert
Reviewed by: no objections on freebsd-arch and freebsd-current
MFC plans: no plans, but will do if people want it in stable.

back out logging to /var/log/adduser
(/usr/sbin/pw already logs to /var/log/userlog)

In interactive mode, ask the user if he/she wants to add another
user and re-run interactive questions or quit depending on the
answer.

Submitted by: Scot Hetzel <hetzels@westbend.net>
Approved by: markm (mentor)

expr foo -> $((foo))

- restore an ability to write log of added users (default to
/var/log/adduser, disabled if empty or adduserlog="no")
- do not ask for password in configure mode
- print $passwdtype instead of password in configure mode
- add DATECMD, GREPCMD (not overridable but with full path)

Do not loose trailing spaces when printing prompt messages.

Do not expose password if it is empty and PWF_STANDARD format is requested

MFC after: 1 week

*blush*
While I'm here properly quote all the other input I neglected
to quote.

Approved by: markm (mentor)(implicit)

Don't forget to properly quote input.

Approved by: markm (mentor)(implicit)
Submitted by: Robin Breathe <robin@isometry.net>

do not loose trailing space when asking uid

Style cleanup.

Make WARNS=5 clean.

Tested on: sparc64, Athlon[32]

english(4) police.

mdoc(7) police: Deal with self-xrefs.

mdoc(7) police: more `The .Nm utility' + markup nit.

Move the NOTES section to DESCRIPTION.  And correct some documentation while
I'm here.

PR: 43756
Submitted by: Gary W. Swearingen <swear@attbi.com>

Uniformly refer to a file system as "file system".

Approved by: re

Correctly handle files with IFS characters (spaces) in their names when
cleaning up temporary files.

Submitted by: Mike Makonnen <mtm@identd.net>
Approved by: re

mdoc(7) police: overhaul.

Approved by: re

Replace the perl versions of adduser and rmuser with shell script versions.

Submitted by: Mike Makonnen <mtm@identd.net>
Approved by: re

mdoc(7) police: markup fixes.

Approved by: re

Clarify the unique groups scheme by mentioning that the 002 umask
is instead of the usual 022 umask, and explain that what the scheme
still prevents is unwanted changes, not prying eyes.

While I'm here, mess with the phrasing and line-breaks a bit.

Don't forget to '\n'-terminate new entries.  This unbreaks chpass -a.

Submitted by: joerg

Don't try to dereference a NULL pw pointer.  This would happen when
attempting to use chpass -a.

Do not allow empty GIDs for non-NIS entries.

MFC after: 2 weeks

Too strict error checking in rev. 1.22 broke pwd_mkdb(8) in NIS
environment. An empty UID and GID are valid there.

Spotted by: rwatson

Disqualify UID/GID with non-numeric character.

PR: bin/41721
Reviewed by: tjr, silence on -audit
MFC after: 2 weeks

Replace various spelling with FALLTHROUGH which is lint()able

Add LOGIN_SETMAC, which will indicate to the user context management code
that it should also set the user's default MAC label, if available and
permitted.

Obtained from: TrustedBSD Project
Sponsored by: DARPA, NAI Labs

Fix conjugation of exists and add full stops to the messages I changed.

PR: 38481
Submitted by: Kevin Kinsey <kadmin@elisha.daleco.biz>
MFC after: 1 week

Whitespace cleanup--it's not style(9), but it is consistent.  Prep
for MAC-related commits to the login infrastructure.

Obtained from: TrustedBSD Project
Sponsored by: DARPA, NAI Labs

The .Nm utility

de-__P()

Consistently use FBSDID

Lists follow a colon, not a semicolon.

`pw useradd' could be used with -w without -D option.

PR: docs/39770
Submitted by: sada
Approved by: dd

Print a long with %ld not %d. (We possibly should be using %j here, but
that can be fixed when the many other warnings here are being fixed.)

PR: 39741
Submitted by: Dan Lukes <dan@obluda.cz>
MFC after: 1 week

Be more clear in error messages.
Distinguish between a held lock and a failed lock op.

If rpc.lockd is not running on a diskless client this makes clearer
what the problem is.

If no old_pw was passed to pw_copy, compare just the name.

Sponsored by: DARPA, NAI Labs

Use libutil and libypclnt for all passwd manipulation and NIS needs.

Sponsored by: DARPA, NAI Labs

Add passwd manipulation code based on parts of vipw and chpass.

Sponsored by: DARPA, NAI Labs

Use `The .Nm utility'

Make mppath and masterpasswd pointers instead of arrays, and initialize
them to point at static strings that contain the default paths.  This
makes 'vipw -d' work again (I broke it in rev 1.21; apologies for taking
so long to fix it.)

Spotted by: Olivier Houchard <doginou@cognet.ci0.org>
Sponsored by: DARPA, NAI Labs

Remove bogus reference to _use_yp.

Const poisoning.

Rename a couple of local variables to avoid warnings.

Sponsored by: DARPA, NAI Labs

In the words of the submitter:

 This patch explains -F for usershow and groupshow.  Because "groupmod
 ... -F" doesn't do anything, the patch also drops that from groupmod's
 command line args.

PR: 35955
Submitted by: Mike DeGraw-Bertsch <mbertsch@radioactivedata.org>

Document the -7 option for usershow.

PR: 36735
Submitted by: Mike DeGraw-Bertsch <mbertsch@radioactivedata.org>

Fix vendor ID.
Make usage message match the man page. (Missing -o, -h doesn't depend on -d).

PR: 36470
Submitted by: Gary W. Swearingen <swear@blarg.net>
MFC after: 1 week

Minor post warns stuff.
1) Fix up vendor IDs for files I'm touching and put FBSDID in the right place.
2) Cast return value of snprintf to size_t rather than casting sizeof() to
   an int, 'cos we know snprintf returns a non-genative number.
3) Avoid 'char *blank = "";' by just strduping. This could be bad sometimes,
   but all these variables usually point at malloced memory so this makes
   them always point at malloced memory.

Fix warns, ANSIfy, use __FBSDID(), sort headers.

Fix a style(9) error. Put the vendor version control string back in.
I nuked it when I added the $FreeBSD$ tag.

Fix the style of the SCM ID's.
I believe have made all of libc .h's as consistent as possible.

Remove multi-line __P() usage.

remove __P

Remove __P() usage.

Remove __P() usage.

Fix a bug that prevents "00" being used as a valid year. Why bother in
2002? It is a bug. Might as well close the PR.

PR: misc/14511
Submitted by: Mike Pritchard <mpp@mpp.pro-ns.net>
MFC after: 3 days

It was possible for an unprivileged user to tie up the password
information (no one else can vipw(8), chpass(1), or even passwd(1)),
either on purpose or by accident, until an administrator manually
intervened. Instead, do not lock the master.passwd file while a user
is editing his information. But once we go to write the new
information, check that the modified user's information has not
changed in the password database since we started. Abort the changes
if it has.

Add a $FreeBSD$ to pw_copy.h.

PR: i386/35816
Obtained from: NetBSD
MFC after: 1 week

Bring a comment in line with the code, and fix some spelling errors
while I'm here.

PR: 35394
Submitted by: andrew@ugh.net.au

Merge a duplicate description of the -L option into the first one.

PR: docs/34782
MFC after: 1 week

Remove a bogus cast.

#include <stddef.h> for the definition of NULL instead of depending on
namespace pollution 2 layers deep in <sys/stat.h>.

Sorted includes.

Found a single point where rmuser(8) wasn't robust to strange
characters in a username:  where it was inserted into a regexp.
Fix it by escaping metacharacters in the name with \Q-\E.

ANSIfy and remove some dead code.

Sponsored by: DARPA, NAI Labs