Use asserts to check validity of pointers in private impl of fileproviderdomainmanager
authorClaudio Cambra <claudio.cambra@nextcloud.com>
Sat, 11 Mar 2023 13:06:20 +0000 (14:06 +0100)
committerClaudio Cambra <claudio.cambra@nextcloud.com>
Fri, 12 May 2023 05:21:28 +0000 (13:21 +0800)
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
src/gui/macOS/fileproviderdomainmanager_mac.mm

index df8a7d3f08b4f69804daf69aeb964139103aa18f..ca96f6edf1568da50a034aa44573c72214eb61f5 100644 (file)
@@ -123,6 +123,10 @@ class FileProviderDomainManager::Private {
 
     void addFileProviderDomain(const AccountState * const accountState)
     {
+        Q_ASSERT(accountState);
+        const auto account = accountState->account();
+        Q_ASSERT(account);
+
         const auto domainDisplayName = domainDisplayNameForAccount(account);
         const auto domainId = domainIdentifierForAccount(account);
 
@@ -150,6 +154,9 @@ class FileProviderDomainManager::Private {
 
     void removeFileProviderDomain(const AccountState * const accountState)
     {
+        Q_ASSERT(accountState);
+        const auto account = accountState->account();
+        Q_ASSERT(account);
 
         const auto domainId = domainIdentifierForAccount(account);
         qCDebug(lcMacFileProviderDomainManager) << "Removing file provider domain with id: " << domainId;
@@ -230,6 +237,7 @@ class FileProviderDomainManager::Private {
 
     void signalEnumeratorChanged(const Account * const account)
     {
+        Q_ASSERT(account);
         const auto domainId = domainIdentifierForAccount(account);
 
         qCDebug(lcMacFileProviderDomainManager) << "Signalling enumerator changed in file provider domain for account with id: " << domainId;