Remove duplicate logging to syslog.

Split check_perms() out of main().

Avoid assignments in conditionals.

Avoid assignments in conditionals.

(split_groups): Test the pointer returned by malloc.

Document add_one_entry_nis(), write_all(), commonio_remove(),
commonio_locate(), and commonio_rewind().

Fix typo s/groupadd/chpasswd/

New TODO item:
 - newusers: add logging to SYSLOG & AUDIT

Fix typo s/groupadd/chage/

Fix typo s/groupadd/chgpasswd/.

Avoid implicit conversions to booleans.

Avoid implicit brackets.

Re-indent.

Avoid implicit conversions to booleans.

use_system_pw_file and use_system_spw_file were not reset.

Re-indent.

Avoid implicit brackets.

ignore libmisc.a

No functional changes were introduced by the previous pwck and grpck
changes, except for the following bug fix: no syslog logging if a passwd
or group file was specified on the command line without a shadowed
database file, even if the system shadowed database was changed).

* Fix open_files prototype (void argument).
* close_file documentation (s:password/shadow:group/gshadow:)

Fix typos in comments (gshadow/shadow).

Simplify pwck's main(). Remove gotos.

Split also check_pw_file() and check_spw_file() out of main().

Also split open_files and close_files out of main().
New global variables use_system_pw_file and use_system_spw_file

Split process_flags() out of main(). New global variables is_shadow,
sort_mode.

De-comment code (duplicate the entry when the _R function is not present on the system).

* src/lastlog.c: Remove statbuf, not used.
* src/lastlog.c: Fix types, cast umin and umax to uid_t.
* src/lastlog.c: (option -u) user needs to be a signed long, not
  uid_t (to accept rangees like -<uid>

Avoid ?: construct without the middle term.

Forgot to commit that one.
Avoid empty file when USE_PAM is set.

* libmisc/copydir.c, src/usermod.c, lib/prototypes.h: The uid and
  gid parameters can be set to -1 to indicate that the original
  owners must be kept. Change the types from uid_t/gid_t to a
  long int (signed).
* libmisc/copydir.c: Change the copy_entry(), copy_dir(),
  copy_symlink(), copy_special(), and copy_file() prototypes
  accordingly.
* lib/prototypes.h: Add the parameters' name for the
  libmisc/copydir.c functions.

Avoid empty file when WITH_AUDIT is not set.

* libmisc/limits.c, libmisc/obscure.c, src/login_nopam.c,
  lib/pwauth.c: Avoid empty file when USE_PAM is set.
* src/login_nopam.c: Fix warnings: resolve_hostname takes and
  returns a constant string.

Remove inaccurate documentation.
The password file is no more opened in read only mode.

Add missing prototype declarations.
Document the new functions.

I forgot to mention compare_members_lists().

Split check_members() out of check_grp_file() and check_sgr_file().

Split check_grp_file() and check_sgr_file() out of main().

Split process_flags(), open_files(), and close_files() out of main(). New
global variables is_shadow, sort_mode, use_system_grp_file, and
use_system_sgr_file.

If remove-potcdate.sin does not exist, use the one from teh po directory
(it is not installed automatically by autopoint.

Compilation fix. pamh needs to be global since the split of check_perms().

Compilation fix. user was removed from the list of global variables.

Fix the type of the bitfields in the commonio_entry and commonio_db
structures to unsigned int (instead of int).

Re-indent.

Re-indent.

Avoid assignments in comparisons.

Avoid implicit brackets.

(compil fix) Use pw->pw_name instead of user.

sflg needs to be global.

Also split update_shell() out of main().

* Split also check_perms() out of main().
* Before pam_end(), the return value of the previous
  pam API was already checked. No need to validate it again.

Split process_flags() out of main().

Avoid assignments in comparisons.

Avoid assignments in comparisons.

Avoid implicit conversions to booleans.

Document may_change_field().

Avoid implicit brackets.

* New function: process_flags() split out of main().
  The flags variables are now global.
* New functions: check_perms(), update_gecos(),
  get_old_fields(), and check_fields() split out of main().
* Before pam_end(), the return value of the previous
  pam API was already checked. No need to validate it again.

* src/newusers.c: Compilation fix for PAM support (pamh needs to be
  global since the function split).
* src/chpasswd.c: Likewise.
* src/chgpasswd.c: Likewise.
* src/chpasswd.c: Avoid implicit conversions to booleans.

Sort the tools in the NEWS entries of 4.1.1.

Rewrote to match the previous commit message.

* src/chage.c: Fix typo: s/maximim/maximum/
* src/chage.c: New function: fail_exit(). Change most of the exit()
to a fail_exit, which makes sure the files are unlocked (new global
variables: pw_locked, spw_locked), the PAM transaction is ended, and
the failure is logged to libaudit (use a global user_name and user_uid
for logging).
* src/chage.c: Compilation fix for PAM support (pamh needs to be
global since the function split).
* src/chage.c: Document process_flags(), check_flags(), check_perms(),
open_files(), and close_files().
* src/chage.c: Split update_age() and get_defaults() out of main()
* src/chage.c: Drop the privileges just after opening the files.
* src/chage.c: Do not log to audit only if the user has an entry in
the shadow file.
* NEWS, src/chage.c (open_files): Also open the password file for
writing. This fix chage when the user only has a password entry (and
no shadow entries).
* src/chage.c (get_defaults): Use default values that don't change the
behavior of the account for the fields that are not specified when the
user has no shadow entry.

* Compilation fix for PAM support (pamh needs to be
  global since the function split).
* End the PAM transaction in fail_exit().
* Document check_flags().

Compilation fix for non-gshadow support.

I forgot to open and close gshadow.

Added support for gshadow.

Do not add the new user to the group's members, because the group is already
the primary group of the new user.

Avoid variables with the name of a type.

Avoid assignments in comparisons.

Avoid implicit brackets and re-indent.

Before pam_end(), the return value of the previous
pam API was already checked. No need to validate it again.

newusers cleanups
main() split in new functions: process_flags(), check_flags(), check_perms(),
open_files(), and close_files().

Avoid assignments in comparisons.

Re-indent.

 * Avoid implicit brackets.
 * Avoid implicit conversion to booleans.

Remove dead code. It was probably put here to add more
information to the audit_logger.

Avoid using a variable with the same name as a type.

chage cleanups
* src/chage.c: Before pam_end(), the return value of the previous
pam API was already checked. No need to validate it again.
* src/chage.c: main() split in new functions: process_flags(),
check_flags(), check_perms(), open_files(), and close_files().

* src/chgpasswd.c: Avoid assignments in comparisons.
* src/chgpasswd.c: Avoid implicit brackets.
* src/chgpasswd.c: Fix comments to match chgpasswd (group instead of
user's passwords are changed).

Fix the previous ChangeLog entries regarding chgpasswd.

Same changes as for chpasswd:
* src/chpasswd.c: main() split in process_flags(), check_flags(),
check_perms(), open_files(), and close_files().

Same changes as for chpasswd:
* src/chpasswd.c: main() split in process_flags(), check_flags(),
check_perms(), open_files(), and close_files().

Avoid implicit brackets.

Re-indent.

Re-indent.

Avoid assignments in comparisons.

Other new functions: open_files(), close_files().
This force flushing the password database after the password file is unlocked.

New functions: process_flags(), check_flags(),
check_perms(). Split out of main().

Before pam_end(), the return value of the previous
pam API was already checked. No need to validate it again.

New function check_flags(). Split the validation of
options and arguments out of process_flags.

(main, check_perms): New function check_perms().
Split the validation of the user's permissions out of main()

(main): Before pam_end(), the return value of the previous pam API was already
checked. No need to validate it again.

Re-indent.

(process_flags): prefer fail_exit to exit. This avoid
an explicit call to audit_logger().

I forgot the initialization of group_id in find_new_gid().

 * process_args renamed process_flags
 * Add the options checks in process_flags (group_name, group ID uniqueness)
 * Add the parameters' names in the prototypes.

Split the processing of options out of main().

find_new_gid is never called when an
GID is specified with -g. Simplify find_new_gid accordingly.

typo cleared/clearer

(find_new_gid): If oflg is set, gflg is also set.
Use (!gflg), which is cleared than (!gflg || !oflg).

A group with the specified name cannot exist at that time in find_new_gid.
Remove the check.

Avoid implict brackets.