From 18a654d13b608f29d559e2c433841cef5e426aa0 Mon Sep 17 00:00:00 2001 From: nekral-guest Date: Fri, 28 Dec 2007 09:39:22 +0000 Subject: [PATCH] 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. --- ChangeLog | 7 +++++++ src/groupadd.c | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 49e89590..251902b3 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2007-12-27 Nicolas François + + src/groupadd.c cleanup + * src/groupadd.c (fail_exit): 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. + 2007-12-27 Nicolas François libmisc/copydir.c cleanup diff --git a/src/groupadd.c b/src/groupadd.c index 4dae0d04..8b47653c 100644 --- a/src/groupadd.c +++ b/src/groupadd.c @@ -346,8 +346,8 @@ static void fail_exit (int code) if (is_shadow_grp) sgr_unlock (); #endif - if (code != E_SUCCESS) #ifdef WITH_AUDIT + if (code != E_SUCCESS) audit_logger (AUDIT_USER_CHAUTHTOK, Prog, "adding group", group_name, -1, 0); #endif