summaryrefslogtreecommitdiff
path: root/loginutils
diff options
context:
space:
mode:
authorDenis Vlasenko2006-09-09 14:00:58 +0000
committerDenis Vlasenko2006-09-09 14:00:58 +0000
commit9852d5a1e65d23dece9bdfa2483409dda258748f (patch)
tree311f3b43614b9fcf0e3adf508b944ac458b8e18d /loginutils
parent8fafacd7ec2ea70cb46b28e59c4c9b7473c44ca9 (diff)
downloadbusybox-9852d5a1e65d23dece9bdfa2483409dda258748f.zip
busybox-9852d5a1e65d23dece9bdfa2483409dda258748f.tar.gz
sulogin: minor cleanup.
Diffstat (limited to 'loginutils')
-rw-r--r--loginutils/sulogin.c32
1 files changed, 15 insertions, 17 deletions
diff --git a/loginutils/sulogin.c b/loginutils/sulogin.c
index 15f3fb2..fef3760 100644
--- a/loginutils/sulogin.c
+++ b/loginutils/sulogin.c
@@ -9,10 +9,6 @@
#include "busybox.h"
-
-#define SULOGIN_PROMPT "Give root password for system maintenance\n" \
- "(or type Control-D for normal startup):"
-
static const char * const forbid[] = {
"ENV",
"BASH_ENV",
@@ -32,7 +28,6 @@ static const char * const forbid[] = {
};
-
static void catchalarm(int ATTRIBUTE_UNUSED junk)
{
exit(EXIT_FAILURE);
@@ -48,10 +43,8 @@ int sulogin_main(int argc, char **argv)
struct passwd *pwd;
struct spwd *spwd;
- if (ENABLE_FEATURE_SYSLOG) {
- logmode = LOGMODE_BOTH;
- openlog(bb_applet_name, LOG_CONS | LOG_NOWAIT, LOG_AUTH);
- }
+ logmode = LOGMODE_BOTH;
+ openlog(bb_applet_name, 0, LOG_AUTH);
if (bb_getopt_ulflags (argc, argv, "t:", &timeout_arg)) {
if (safe_strtoi(timeout_arg, &timeout)) {
@@ -68,7 +61,8 @@ int sulogin_main(int argc, char **argv)
}
if (!isatty(0) || !isatty(1) || !isatty(2)) {
- bb_error_msg_and_die("Not a tty");
+ logmode = LOGMODE_SYSLOG;
+ bb_error_msg_and_die("not a tty");
}
/* Clear out anything dangerous from the environment */
@@ -78,28 +72,31 @@ int sulogin_main(int argc, char **argv)
signal(SIGALRM, catchalarm);
if (!(pwd = getpwuid(0))) {
- goto AUTH_ERROR;
+ goto auth_error;
}
if (ENABLE_FEATURE_SHADOWPASSWDS) {
if (!(spwd = getspnam(pwd->pw_name))) {
- goto AUTH_ERROR;
+ goto auth_error;
}
pwd->pw_passwd = spwd->sp_pwdp;
}
while (1) {
/* cp points to a static buffer that is zeroed every time */
- cp = bb_askpass(timeout, SULOGIN_PROMPT);
+ cp = bb_askpass(timeout,
+ "Give root password for system maintenance\n"
+ "(or type Control-D for normal startup):");
+
if (!cp || !*cp) {
bb_info_msg("Normal startup");
- exit(EXIT_SUCCESS);
+ return 0;
}
if (strcmp(pw_encrypt(cp, pwd->pw_passwd), pwd->pw_passwd) == 0) {
break;
}
bb_do_delay(FAIL_DELAY);
- bb_error_msg("Login incorrect");
+ bb_error_msg("login incorrect");
}
memset(cp, 0, strlen(cp));
signal(SIGALRM, SIG_DFL);
@@ -110,6 +107,7 @@ int sulogin_main(int argc, char **argv)
run_shell(pwd->pw_shell, 1, 0, 0);
/* never returns */
-AUTH_ERROR:
- bb_error_msg_and_die("No password entry for `root'");
+
+auth_error:
+ bb_error_msg_and_die("no password entry for `root'");
}