diff options
author | Karel Zak | 2012-12-12 10:41:44 +0100 |
---|---|---|
committer | Karel Zak | 2012-12-12 10:41:44 +0100 |
commit | 6affea028b655ebf8d6dc2181d4a2f94424153ac (patch) | |
tree | 9e6da1948755bc05d535d69170c91973a2e53683 /login-utils/chfn.c | |
parent | chfn: clean up exit status (diff) | |
download | kernel-qcow2-util-linux-6affea028b655ebf8d6dc2181d4a2f94424153ac.tar.gz kernel-qcow2-util-linux-6affea028b655ebf8d6dc2181d4a2f94424153ac.tar.xz kernel-qcow2-util-linux-6affea028b655ebf8d6dc2181d4a2f94424153ac.zip |
chfn: always use return rather than exit() in main
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'login-utils/chfn.c')
-rw-r--r-- | login-utils/chfn.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/login-utils/chfn.c b/login-utils/chfn.c index a86f380c0..76c0b03ee 100644 --- a/login-utils/chfn.c +++ b/login-utils/chfn.c @@ -165,22 +165,22 @@ int main(int argc, char **argv) retcode = pam_start("chfn", oldf.username, &conv, &pamh); if (pam_fail_check(pamh, retcode)) - exit(EXIT_FAILURE); + return EXIT_FAILURE; retcode = pam_authenticate(pamh, 0); if (pam_fail_check(pamh, retcode)) - exit(EXIT_FAILURE); + return EXIT_FAILURE; retcode = pam_acct_mgmt(pamh, 0); if (retcode == PAM_NEW_AUTHTOK_REQD) retcode = pam_chauthtok(pamh, PAM_CHANGE_EXPIRED_AUTHTOK); if (pam_fail_check(pamh, retcode)) - exit(EXIT_FAILURE); + return EXIT_FAILURE; retcode = pam_setcred(pamh, 0); if (pam_fail_check(pamh, retcode)) - exit(EXIT_FAILURE); + return EXIT_FAILURE; pam_end(pamh, 0); /* no need to establish a session; this isn't a |