summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorDenis Vlasenko <vda.linux@googlemail.com>2006-09-23 13:11:49 +0000
committerDenis Vlasenko <vda.linux@googlemail.com>2006-09-23 13:11:49 +0000
commita36a676923a674e35aae65faaa169fab9d765023 (patch)
treec99eb90ceea7adfebc157ca28a45cf9ae81b4a57
parent7f1f5b001f5a0dee058024f7bee04dced38877df (diff)
correct_password: undo whitespace damage.
vlock + correct_password: fix incorrect line breaks in messages.
-rw-r--r--libbb/correct_password.c38
-rw-r--r--loginutils/su.c2
-rw-r--r--loginutils/vlock.c7
3 files changed, 17 insertions, 30 deletions
diff --git a/libbb/correct_password.c b/libbb/correct_password.c
index 527b3100b..fd7e0b56c 100644
--- a/libbb/correct_password.c
+++ b/libbb/correct_password.c
@@ -28,50 +28,38 @@
* SUCH DAMAGE.
*/
-#include <stdio.h>
-#include <errno.h>
-#include <unistd.h>
-#include <string.h>
-#include <stdlib.h>
-#include <syslog.h>
-#include <ctype.h>
-#include <crypt.h>
-
#include "libbb.h"
-
-
/* Ask the user for a password.
Return 1 if the user gives the correct password for entry PW,
0 if not. Return 1 without asking for a password if run by UID 0
or if PW has an empty password. */
-int correct_password ( const struct passwd *pw )
+int correct_password(const struct passwd *pw)
{
char *unencrypted, *encrypted, *correct;
#ifdef CONFIG_FEATURE_SHADOWPASSWDS
- if (( strcmp ( pw-> pw_passwd, "x" ) == 0 ) || ( strcmp ( pw-> pw_passwd, "*" ) == 0 )) {
- struct spwd *sp = getspnam ( pw-> pw_name );
+ if (!strcmp(pw->pw_passwd, "x") || !strcmp(pw->pw_passwd, "*")) {
+ struct spwd *sp = getspnam(pw->pw_name);
- if ( !sp )
- bb_error_msg_and_die ( "\nno valid shadow password" );
+ if (!sp)
+ bb_error_msg_and_die("no valid shadow password");
- correct = sp-> sp_pwdp;
+ correct = sp->sp_pwdp;
}
else
#endif
- correct = pw-> pw_passwd;
+ correct = pw->pw_passwd;
- if ( correct == 0 || correct[0] == '\0' )
+ if (!correct || correct[0] == '\0')
return 1;
- unencrypted = bb_askpass ( 0, "Password: " );
- if ( !unencrypted )
- {
+ unencrypted = bb_askpass(0, "Password: ");
+ if (!unencrypted) {
return 0;
}
- encrypted = crypt ( unencrypted, correct );
- memset ( unencrypted, 0, strlen ( unencrypted ));
- return ( strcmp ( encrypted, correct ) == 0 ) ? 1 : 0;
+ encrypted = crypt(unencrypted, correct);
+ memset(unencrypted, 0, strlen(unencrypted));
+ return (!strcmp(encrypted, correct)) ? 1 : 0;
}
diff --git a/loginutils/su.c b/loginutils/su.c
index ff6643e60..c51359ae1 100644
--- a/loginutils/su.c
+++ b/loginutils/su.c
@@ -45,7 +45,7 @@ int su_main(int argc, char **argv)
}
pw = getpwnam(opt_username);
- if (!pw) bb_error_msg_and_die("Unknown id: %s", opt_username);
+ if (!pw) bb_error_msg_and_die("unknown id: %s", opt_username);
/* Make sure pw->pw_shell is non-NULL. It may be NULL when NEW_USER
is a username that is retrieved via NIS (YP), but that doesn't have
diff --git a/loginutils/vlock.c b/loginutils/vlock.c
index b4426ad41..564b6d527 100644
--- a/loginutils/vlock.c
+++ b/loginutils/vlock.c
@@ -108,13 +108,12 @@ int vlock_main(int argc, char **argv)
tcsetattr(STDIN_FILENO, TCSANOW, &term);
do {
- printf("Virtual Console%s locked.\n%s's ", (o_lock_all) ? "s" : "", pw->pw_name);
- fflush(stdout);
- if (correct_password (pw)) {
+ printf("Virtual Console%s locked. Enter %s's password to unlock\n", (o_lock_all) ? "s" : "", pw->pw_name);
+ if (correct_password(pw)) {
break;
}
bb_do_delay(FAIL_DELAY);
- puts("Password incorrect.");
+ puts("Password incorrect");
} while (1);
restore_terminal();
return 0;