correct_password: do not print "no shadow passwd..." message
authorDenis Vlasenko <vda.linux@googlemail.com>
Tue, 3 Jul 2007 06:15:42 +0000 (06:15 -0000)
committerDenis Vlasenko <vda.linux@googlemail.com>
Tue, 3 Jul 2007 06:15:42 +0000 (06:15 -0000)
function                                             old     new   delta
correct_password                                     204     191     -13
.rodata                                           129530  129466     -64
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-77)             Total: -77 bytes
   text    data     bss     dec     hex filename
 675984    2744   13968  692696   a91d8 busybox_old
 675908    2744   13968  692620   a918c busybox_unstripped

libbb/correct_password.c

index f832635e6b0331a08cfcf8d19c2f55cb13be0230..815c51c43f5af2ac75116b12c976ccfd5a9ed469 100644 (file)
@@ -40,12 +40,6 @@ int correct_password(const struct passwd *pw)
 {
        char *unencrypted, *encrypted;
        const char *correct;
-#if ENABLE_FEATURE_SHADOWPASSWDS
-       /* Using _r function to avoid pulling in static buffers */
-       struct spwd spw;
-       struct spwd *result;
-       char buffer[256];
-#endif
 
        /* fake salt. crypt() can choke otherwise. */
        correct = "aa";
@@ -55,11 +49,14 @@ int correct_password(const struct passwd *pw)
        }
        correct = pw->pw_passwd;
 #if ENABLE_FEATURE_SHADOWPASSWDS
-       if (LONE_CHAR(pw->pw_passwd, 'x') || LONE_CHAR(pw->pw_passwd, '*')) {
-               if (getspnam_r(pw->pw_name, &spw, buffer, sizeof(buffer), &result))
-                       bb_error_msg("no valid shadow password, checking ordinary one");
-               else
+       if ((correct[0] == 'x' || correct[0] == '*') && !correct[1]) {
+               /* Using _r function to avoid pulling in static buffers */
+               struct spwd spw;
+               struct spwd *result;
+               char buffer[256];
+               if (getspnam_r(pw->pw_name, &spw, buffer, sizeof(buffer), &result) == 0)
                        correct = spw.sp_pwdp;
+               /* else: no valid shadow password, checking ordinary one */
        }
 #endif