From 210176fd915cf4ba16f428d3c1a249a71f4aa7a7 Mon Sep 17 00:00:00 2001 From: Treeston Date: Sun, 26 Jul 2020 01:53:34 +0200 Subject: Core/Authserver: Authserver cleanup (PR#25093) - Fix a handful of 1/256 bugs with most significant byte zero in BigNumber - Get rid of (most of) the C-style arrays in authserver - CryptoRandom as a unified source for cryptographic randomness - Bring our other crypto APIs into 2020 - BigNumber usability improvements - Authserver is now actually readable as a result of all of the above --- src/server/shared/Secrets/SecretMgr.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/server/shared/Secrets/SecretMgr.cpp') diff --git a/src/server/shared/Secrets/SecretMgr.cpp b/src/server/shared/Secrets/SecretMgr.cpp index 590440973f5..a0180ae2363 100644 --- a/src/server/shared/Secrets/SecretMgr.cpp +++ b/src/server/shared/Secrets/SecretMgr.cpp @@ -190,13 +190,13 @@ Optional SecretMgr::AttemptTransition(Secrets i, Optional(totpSecret, oldSecret->AsByteArray()); + bool success = Trinity::Crypto::AEDecrypt(totpSecret, oldSecret->ToByteArray()); if (!success) return Trinity::StringFormat("Cannot decrypt old TOTP tokens - value of '%s' is incorrect for some users!", secret_info[i].oldKey); } if (newSecret) - Trinity::Crypto::AEEncryptWithRandomIV(totpSecret, newSecret->AsByteArray()); + Trinity::Crypto::AEEncryptWithRandomIV(totpSecret, newSecret->ToByteArray()); LoginDatabasePreparedStatement* updateStmt = LoginDatabase.GetPreparedStatement(LOGIN_UPD_ACCOUNT_TOTP_SECRET); updateStmt->setBinary(0, totpSecret); -- cgit v1.2.3