Merge pull request #20 from neo-project/master

Fix change password (neo-project#1490)
Qiao-Jin · Mar 23, 2020 · 8988eda · 8988eda
2 parents 33e0e26 + 6eba2ee
commit 8988eda
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/neo/Wallets/SQLite/UserWallet.cs b/src/neo/Wallets/SQLite/UserWallet.cs
@@ -174,7 +174,7 @@ public bool ChangePassword(string password_old, string password_new)
             byte[] passwordKey = password_new.ToAesKey();
             try
             {
-                SaveStoredData("PasswordHash", passwordKey.Sha256());
+                SaveStoredData("PasswordHash", passwordKey.Concat(salt).ToArray().Sha256());
                 SaveStoredData("MasterKey", masterKey.AesEncrypt(passwordKey, iv));
                 return true;
             }

diff --git a/tests/neo.UnitTests/Wallets/SQLite/UT_UserWallet.cs b/tests/neo.UnitTests/Wallets/SQLite/UT_UserWallet.cs
@@ -167,6 +167,7 @@ public void TestChangePassword()
         {
             wallet.ChangePassword("123455", "654321").Should().BeFalse();
             wallet.ChangePassword("123456", "654321").Should().BeTrue();
+            wallet.ChangePassword("654321", "123456").Should().BeTrue();
         }
 
         [TestMethod]