summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFabian Bläse <fabian@blaese.de>2020-03-17 16:55:57 +0100
committerJo-Philipp Wich <jo@mein.io>2020-03-17 19:33:35 +0100
commit3df62bcebd2caff47506ccffb42df8d2d2123c06 (patch)
tree921be69d822989ad3efe7d13ae32949e52148975
parentefe51f41d6288fa8503cc4f2efb37a777767d505 (diff)
downloadrpcd-3df62bcebd2caff47506ccffb42df8d2d2123c06.tar.gz
session: deny access if password login is disabled
The special characters '!' and '*' in /etc/shadow are used to disable password login for a specific account. The character 'x' has no special meaning, but should not be interpreted as an empty password. However, rpcd did treat these special characters like no password was set, which allows access even though the account is disabled. By removing the additional checks for these characters, the encrypted password string is passed to crypt, which returns NULL if the salt has an invalid format and therefore access is denied. Fixes: FS#2634 Signed-off-by: Fabian Bläse <fabian@blaese.de>
-rw-r--r--session.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/session.c b/session.c
index 13a2ef3..908e298 100644
--- a/session.c
+++ b/session.c
@@ -795,7 +795,7 @@ rpc_login_test_password(const char *hash, const char *password)
char *crypt_hash;
/* password is not set */
- if (!hash || !*hash || !strcmp(hash, "!") || !strcmp(hash, "x"))
+ if (!hash || !*hash)
{
return true;
}