From: Denis Vlasenko Date: Mon, 27 Nov 2006 01:04:23 +0000 (-0000) Subject: deluser: the same code, but a bit less obfuscated. X-Git-Tag: 1_4_0~313 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a7b2c41e03555a9f7a7d44e80f65518b167fa280;p=platform%2Fupstream%2Fbusybox.git deluser: the same code, but a bit less obfuscated. --- diff --git a/loginutils/deluser.c b/loginutils/deluser.c index e337101..869b220 100644 --- a/loginutils/deluser.c +++ b/loginutils/deluser.c @@ -34,28 +34,29 @@ static void del_line_matching(const char *login, const char *filename) } } - if (!found) { - bb_error_msg("can't find '%s' in '%s'", login, filename); - if (!ENABLE_FEATURE_CLEAN_UP) return; - goto clean_up; - } - - if (ENABLE_FEATURE_CLEAN_UP) + if (!ENABLE_FEATURE_CLEAN_UP) { + if (!found) { + bb_error_msg("can't find '%s' in '%s'", login, filename); + return; + } + passwd = fopen_or_warn(filename, "w"); + if (passwd) + while ((line = llist_pop(&plist))) + fputs(line, passwd); + } else { + if (!found) { + bb_error_msg("can't find '%s' in '%s'", login, filename); + goto clean_up; + } fclose(passwd); - - passwd = fopen_or_warn(filename, "w"); - if (passwd) { - if (ENABLE_FEATURE_CLEAN_UP) { + passwd = fopen_or_warn(filename, "w"); + if (passwd) { clean_up: while ((line = llist_pop(&plist))) { if (found) fputs(line, passwd); free(line); } fclose(passwd); - } else { - /* found != 0 here, no need to check */ - while ((line = llist_pop(&plist))) - fputs(line, passwd); } } }