mirrors/nextcloud-desktop
1
0
Fork 0
mirror of https://github.com/nextcloud/desktop.git synced 2024-10-24 05:15:40 +03:00
Code Issues Projects Releases Packages Wiki Activity

Assert validity of accountstate and its accountptr when adding it

Browse source 
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
This commit is contained in:
Claudio Cambra 2024-06-25 18:22:30 +08:00
parent 74a78242e2
commit 1689f7b613
2 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/gui/accountmanager.cpp
View file

@ -568,11 +568,12 @@ QString AccountManager::generateFreeAccountId() const
}
}
void AccountManager::addAccountState(AccountState *accountState)
void AccountManager::addAccountState(AccountState *const accountState)
{
QObject::connect(accountState->account().data(),
&Account::wantsAccountSaved,
this, &AccountManager::saveAccount);
Q_ASSERT(accountState);
Q_ASSERT(accountState->account());
QObject::connect(accountState->account().data(), &Account::wantsAccountSaved, this, &AccountManager::saveAccount);
AccountStatePtr ptr(accountState);
_accounts << ptr;

2
src/gui/accountmanager.h
View file

@ -126,7 +126,7 @@ private:
[[nodiscard]] QString generateFreeAccountId() const;
// Adds an account to the tracked list, emitting accountAdded()
void addAccountState(AccountState *accountState);
void addAccountState(AccountState *const accountState);
AccountManager() = default;
QList<AccountStatePtr> _accounts;