summaryrefslogtreecommitdiff
path: root/loginutils/deluser.c
diff options
context:
space:
mode:
authorHarald Becker2010-09-22 14:50:15 +0200
committerDenys Vlasenko2010-09-22 18:03:29 +0200
commit361b1a6884bc9d4f5cece22d0eb9fe5eae17bda7 (patch)
tree7657b775df392a70b8dc1ea2bffddef112f71b0d /loginutils/deluser.c
parent0e13b4019c3d05933ba8d37357023bfcd82e8106 (diff)
downloadbusybox-361b1a6884bc9d4f5cece22d0eb9fe5eae17bda7.zip
busybox-361b1a6884bc9d4f5cece22d0eb9fe5eae17bda7.tar.gz
deluser: make it simpler, fix inability to delete user from group
Also makes menuconfig order more sensible Signed-off-by: Harald Becker <ralda@gmx.de> Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
Diffstat (limited to 'loginutils/deluser.c')
-rw-r--r--loginutils/deluser.c54
1 files changed, 26 insertions, 28 deletions
diff --git a/loginutils/deluser.c b/loginutils/deluser.c
index e234d66..65817b9 100644
--- a/loginutils/deluser.c
+++ b/loginutils/deluser.c
@@ -6,25 +6,18 @@
* Copyright (C) 1999,2000,2001 by John Beppu <beppu@codepoet.org>
* Copyright (C) 2007 by Tito Ragusa <farmatito@tiscali.it>
*
- * Licensed under GPLv2, see file LICENSE in this source tree.
+ * Licensed under GPL version 2, see file LICENSE in this tarball for details.
*
*/
#include "libbb.h"
-static int del_line_matching(char **args, const char *filename)
-{
- if (ENABLE_FEATURE_DEL_USER_FROM_GROUP && args[2]) {
- return update_passwd(filename, args[2], NULL, args[1]);
- }
- return update_passwd(filename, args[1], NULL, NULL);
-}
-
int deluser_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
int deluser_main(int argc, char **argv)
{
if (argc != 2
&& (!ENABLE_FEATURE_DEL_USER_FROM_GROUP
- || (applet_name[3] != 'g' || argc != 3))
+ || applet_name[3] != 'g'
+ || argc != 3)
) {
bb_show_usage();
}
@@ -32,25 +25,30 @@ int deluser_main(int argc, char **argv)
if (geteuid())
bb_error_msg_and_die(bb_msg_perm_denied_are_you_root);
- if ((ENABLE_FEATURE_DEL_USER_FROM_GROUP && argc != 3)
- || ENABLE_DELUSER
- || (ENABLE_DELGROUP && ENABLE_DESKTOP)
- ) {
- if (ENABLE_DELUSER
- && (!ENABLE_DELGROUP || applet_name[3] == 'u')
- ) {
- if (del_line_matching(argv, bb_path_passwd_file) < 0)
+ if (ENABLE_DELUSER && applet_name[3] == 'u') {
+ /* deluser USER */
+ if (update_passwd(bb_path_passwd_file, argv[1], NULL, NULL) < 0)
+ return EXIT_FAILURE;
+ if (ENABLE_FEATURE_SHADOWPASSWDS)
+ if (update_passwd(bb_path_shadow_file, argv[1], NULL, NULL) < 0)
return EXIT_FAILURE;
- if (ENABLE_FEATURE_SHADOWPASSWDS) {
- del_line_matching(argv, bb_path_shadow_file);
- }
- } else if (ENABLE_DESKTOP && ENABLE_DELGROUP && getpwnam(argv[1]))
- bb_error_msg_and_die("can't remove primary group of user %s", argv[1]);
- }
- if (del_line_matching(argv, bb_path_group_file) < 0)
- return EXIT_FAILURE;
- if (ENABLE_FEATURE_SHADOWPASSWDS) {
- del_line_matching(argv, bb_path_gshadow_file);
+ } else if (ENABLE_DELGROUP) {
+ /* delgroup ... */
+ if (!ENABLE_FEATURE_DEL_USER_FROM_GROUP || argc != 3) {
+ /* delgroup GROUP */
+ if (update_passwd(bb_path_group_file, argv[1], NULL, NULL) < 0)
+ return EXIT_FAILURE;
+ if (ENABLE_FEATURE_SHADOWPASSWDS)
+ if (update_passwd(bb_path_gshadow_file, argv[1], NULL, NULL) < 0)
+ return EXIT_FAILURE;
+ } else {
+ /* delgroup USER GROUP */
+ if (update_passwd(bb_path_group_file, argv[2], NULL, argv[1]) < 0)
+ return EXIT_FAILURE;
+ if (ENABLE_FEATURE_SHADOWPASSWDS)
+ if (update_passwd(bb_path_gshadow_file, argv[2], NULL, argv[1]) < 0)
+ return EXIT_FAILURE;
+ }
}
return EXIT_SUCCESS;
}