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.

When compiled without AUDIT support, if the return code was E_SUCCESS,
fail_exit() wouldn't have exited. Fix the scope of #idef WITH_AUDIT.

Document the new functions.

Other cleanups and documentation.
Do the checks, then build the filenames. Do not mix both.

Stop at the first error.

Avoid assignement in comparison.

Avoid implicit conversions to booleans.

Avoid implicit casts.

Avoid implicit brackets.

Document selinux_file_context.

Avoid assignment in comparisons.

libmisc/copydir.c cleanup
* libmisc/copydir.c: Split copy_tree() in more maintainable functions:
copy_entry(), copy_dir(), copy_symlink(), copy_hardlink(),
copy_special(), and copy_file().
* libmisc/copydir.c: -1 is used to indicate an error, directly set err
to -1, instead of incrementing it, and checking if not nul at the
end.

Avoid implicit conversions to booleans.

Avoid assignment in comparisons.

Document check_list's return value.

Avoid implicit brackets.

Simplify gpasswd's main():
Also split check_flags() out of main().

Simplify gpasswd's main():
Split also get_group() and change_passwd() out of main().

Simplify gpasswd's main():
New function: check_perms(). Split out of main() to simplify main().

Simplify gpasswd's main():
New functions: open_files(), close_files(), update_group(). Split out
from main() to simplify this (too) big function.

Simplify gpasswd's main():
New function: process_flags(). Split the processing of options out of main().

gpasswd cleanup
* src/gpasswd.c: Add argument name to the internal function
prototypes.
* src/gpasswd.c: Document global variables.