From f848a19d352121a00b8526ed5c2ec60330e25ab2 Mon Sep 17 00:00:00 2001 From: Richard Ipsum Date: Sun, 28 May 2017 19:36:01 +0100 Subject: Add TRUST_LIBSCRYPT_SALT_GEN build option Modern versions of libscrypt now generate salt correctly, indeed using the very method currently used by lua-scrypt.[1] This patch adds a build option that is disabled by default, when enabled lua-scrypt will use libscrypt's salt generation code rather than its own. [1]: https://sources.debian.net/src/libscrypt/1.21-3/crypto-scrypt-saltgen.c/ --- luascrypt.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) (limited to 'luascrypt.c') diff --git a/luascrypt.c b/luascrypt.c index 181f1e8..9fad808 100644 --- a/luascrypt.c +++ b/luascrypt.c @@ -34,7 +34,7 @@ luascrypt_salt_gen(char *salt, int saltlen) * if we can... */ libscrypt_salt_gen(salt, saltlen); - + fd = open("/dev/urandom", O_RDONLY); if (fd >= 0) { read(fd, salt, saltlen); /* Ignore errors in these two calls */ @@ -70,8 +70,15 @@ luascrypt_hash_password(lua_State *L) return luaL_error(L, "Unable to generate password hash: %s", "N is too large (limited to 2^15)"); } - + +#ifdef TRUST_LIBSCRYPT_SALT_GEN + /* Modern versions of libscrypt generate sufficiently random salts + * and take a uint8_t * instead of char * + */ + libscrypt_salt_gen((uint8_t *) salt, sizeof(salt)); +#else luascrypt_salt_gen(salt, sizeof(salt)); +#endif if (libscrypt_scrypt((uint8_t*)passwd, passwd_len, (uint8_t*)salt, sizeof(salt), -- cgit v1.2.1