Don't check errno of getpwnam() and getspnam() functions. They don't seem to

be set correctly in all systems and it's unlikely the error is anything else
than "user doesn't exist".

--HG--
branch : HEAD

diff --git a/src/auth/passdb-passwd.c b/src/auth/passdb-passwd.c
index 34350e8b83374919fe2795833d3182fc9a3cc039..3d2b7bfd5d5ad7501017c1307dbf2e65d076d4d7 100644 (file)
--- a/src/auth/passdb-passwd.c
+++ b/src/auth/passdb-passwd.c
@@ -21,9 +21,7 @@ passwd_verify_plain(struct auth_request *request, const char *password,
 
        pw = getpwnam(request->user);
        if (pw == NULL) {
-               if (errno != 0)
-                       i_error("getpwnam(%s) failed: %m", request->user);
-               else if (verbose)
+               if (verbose)
                        i_info("passwd(%s): unknown user", request->user);
                callback(PASSDB_RESULT_USER_UNKNOWN, request);
                return;

diff --git a/src/auth/passdb-shadow.c b/src/auth/passdb-shadow.c
index d94e9f7968aabdbac7f6e6ee1ec70400bacfdd49..ee473f361a46c95d69a0a4b900e1555db4924ed4 100644 (file)
--- a/src/auth/passdb-shadow.c
+++ b/src/auth/passdb-shadow.c
@@ -21,9 +21,7 @@ shadow_verify_plain(struct auth_request *request, const char *password,
 
        spw = getspnam(request->user);
        if (spw == NULL) {
-               if (errno != 0)
-                       i_error("getspnam(%s) failed: %m", request->user);
-               else if (verbose)
+               if (verbose)
                        i_info("shadow(%s): unknown user", request->user);
                callback(PASSDB_RESULT_USER_UNKNOWN, request);
                return;

diff --git a/src/auth/userdb-passwd.c b/src/auth/userdb-passwd.c
index 601463d20241d371b83da32cdc7e7142ce348fcc..901d9b8d5936ac59a6481a340fa28575d9a11e23 100644 (file)
--- a/src/auth/userdb-passwd.c
+++ b/src/auth/userdb-passwd.c
@@ -18,9 +18,7 @@ static void passwd_lookup(const char *user, userdb_callback_t *callback,
 
        pw = getpwnam(user);
        if (pw == NULL) {
-               if (errno != 0)
-                       i_error("getpwnam(%s) failed: %m", user);
-               else if (verbose)
+               if (verbose)
                        i_info("passwd(%s): unknown user", user);
                callback(NULL, context);
                return;