Fix password salt not stored, generating a new salt everytime

2 files changed, 44 insertions, 0 deletions

diff --git a/src/DatabaseStorage.cpp b/src/DatabaseStorage.cpp
index 14b74e8..de9d0df 100644
--- a/src/DatabaseStorage.cpp
+++ b/src/DatabaseStorage.cpp
@@ -158,6 +158,7 @@ namespace odhtdb
             metadataSerializer.add(STORAGE_VERSION);
             randombytes_buf(passwordSalt, PASSWORD_SALT_LEN);
             metadataSerializer.add(passwordSalt, PASSWORD_SALT_LEN);
+            fileOverwrite(metadataFilePath, { metadataSerializer.getBuffer().data(), metadataSerializer.getBuffer().size() });
         }
         catch(sibs::DeserializeException &e)
         {
diff --git a/tests/main.cpp b/tests/main.cpp
index 1541d2a..988309c 100644
--- a/tests/main.cpp
+++ b/tests/main.cpp
@@ -377,6 +377,48 @@ static void __attribute__((optimize("O0"))) testMemoryLeak()
     }
 }
 
+static void testStoreAccount()
+{
+    boost::filesystem::path storagePath("/tmp/odhtdb");
+    boost::filesystem::remove_all(storagePath);
+    boost::filesystem::create_directory(storagePath);
+
+    auto createNodeCallback = [](const DatabaseCreateNodeRequest &request)
+    {
+        Log::debug("Create node callback");
+    };
+    
+    auto addNodeCallback = [](const DatabaseAddNodeRequest &request)
+    {
+        Log::debug("Add node callback");
+    };
+    
+    auto addUserCallback = [](const DatabaseAddUserRequest &request)
+    {
+        Log::debug("Add user callback");
+    };
+
+    {
+        DatabaseCallbackFuncs callbackFuncs { createNodeCallback, addNodeCallback, addUserCallback };
+        Database database("127.0.0.1", PORT, storagePath, callbackFuncs);
+
+        std::string username = "username";
+        std::string password = "password";
+
+        database.storeUserWithoutNodes(username, password);
+        database.getStoredNodeUserInfoDecrypted(username, password);
+    }
+    {
+        DatabaseCallbackFuncs callbackFuncs { createNodeCallback, addNodeCallback, addUserCallback };
+        Database database("127.0.0.1", PORT, storagePath, callbackFuncs);
+
+        std::string username = "username";
+        std::string password = "password";
+
+        database.getStoredNodeUserInfoDecrypted(username, password);
+    }
+}
+
 struct Test
 {
     function<void()> testFunc;
@@ -394,6 +436,7 @@ int main(int argc, char **argv)
     testByName["two_local_nodes"] = { testTwoLocalNodes, false };
     testByName["memory_usage"] = { testMemoryUsage, true };
     testByName["test_memory_leak"] = { testMemoryLeak, true };
+    testByName["store_account"] = { testStoreAccount, false };
     
     const char *testName = "all";
     if(argc > 1)