Remove check narrowing todos, they are handled
This commit is contained in:
2 files changed
+4
-4
No files matched your search
+1
-1
@@ -64,5 +64,5 @@ class PasswordHashTest {
|
||||
//TODO strNeedsRehash -1 case?
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
}
|
||||
+3
-3
@@ -29,7 +29,7 @@ actual object PasswordHash {
|
||||
outputLength,
|
||||
password.encodeToUByteArray().toUInt8Array(),
|
||||
salt.toUInt8Array(),
|
||||
opsLimit.toInt(), // TODO check this narrowing
|
||||
opsLimit.toInt(),
|
||||
memLimit,
|
||||
algorithm
|
||||
).toUByteArray()
|
||||
@@ -51,7 +51,7 @@ actual object PasswordHash {
|
||||
}
|
||||
return getSodium().crypto_pwhash_str(
|
||||
password.encodeToUByteArray().toUInt8Array(),
|
||||
opslimit.toInt(), // TODO check this narrowing
|
||||
opslimit.toInt(),
|
||||
memlimit
|
||||
).encodeToUByteArray()
|
||||
}
|
||||
@@ -73,7 +73,7 @@ actual object PasswordHash {
|
||||
return if (
|
||||
getSodium().crypto_pwhash_str_needs_rehash(
|
||||
passwordHash.asByteArray().decodeToString(),
|
||||
opslimit.toInt(), // TODO check this narrowing
|
||||
opslimit.toInt(),
|
||||
memlimit
|
||||
)
|
||||
) {
|
||||
|
||||
Reference in new issue
Block a user