(process_flags): prefer fail_exit to exit. This avoid

an explicit call to audit_logger().
This commit is contained in:
nekral-guest 2007-12-28 19:15:14 +00:00
parent da37da30e1
commit ffa34c5afd
2 changed files with 4 additions and 6 deletions

View File

@ -6,6 +6,8 @@
Use (!gflg), which is clearer than (!gflg || !oflg).
* src/groupadd.c (find_new_gid): find_new_gid is never called when an
GID is specified with -g. Simplify find_new_gid accordingly.
* src/groupadd.c (process_flags): prefer fail_exit to exit. This avoid
an explicit call to audit_logger().
2007-12-28 Nicolas François <nicolas.francois@centraliens.net>

View File

@ -453,14 +453,10 @@ static void process_flags (int argc, char **argv)
/* The group already exist */
if (fflg) {
/* OK, no need to do anything */
exit (E_SUCCESS);
fail_exit (E_SUCCESS);
}
fprintf (stderr, _("%s: group %s exists\n"), Prog, group_name);
#ifdef WITH_AUDIT
audit_logger (AUDIT_USER_CHAUTHTOK, Prog,
"adding group", group_name, -1, 0);
#endif
exit (E_NAME_IN_USE);
fail_exit (E_NAME_IN_USE);
}
if (gflg && (getgrgid (group_id) != NULL)) {