GroupDataProvider: PersistentStorageDelegate Initialization (#16079)

* Group Data Provider: PersistentStorageDelegate passed as init() argument. * Group Data Provider: PersistentStorageDelegate: Review comments applied. * Remove storage dependency on GroupDataProvider.h * restyled * Fix comment typo. * Group Data Provider initialization: Rebased. Co-authored-by: Tennessee Carmel-Veilleux <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]>
project-chip · Oct 2, 2023 · 1058220 · 1058220
1 parent 0038993
commit 1058220
Show file tree

Hide file tree

Showing 7 changed files with 80 additions and 59 deletions.
diff --git a/src/app/InteractionModelEngine.cpp b/src/app/InteractionModelEngine.cpp
@@ -176,7 +176,7 @@ uint32_t InteractionModelEngine::GetNumActiveWriteHandlers() const
 }
 
 void InteractionModelEngine::CloseTransactionsFromFabricIndex(FabricIndex aFabricIndex)
-{ 
+{
     //
     // Walk through all existing subscriptions and shut down those whose subscriber matches
     // that which just came in.

diff --git a/src/app/server/Server.cpp b/src/app/server/Server.cpp
@@ -105,8 +105,7 @@ Server::Server() :
         .dnsCache          = nullptr,
 #endif
         .devicePool        = &mDevicePool,
-    }),
-    mGroupsProvider(mDeviceStorage)
+    })
 {}
 
 CHIP_ERROR Server::Init(AppDelegate * delegate, uint16_t secureServicePort, uint16_t unsecureServicePort,
@@ -141,6 +140,7 @@ CHIP_ERROR Server::Init(AppDelegate * delegate, uint16_t secureServicePort, uint
     app::DnssdServer::Instance().SetCommissioningModeProvider(&mCommissioningWindowManager);
 
     // Group data provider must be initialized after mDeviceStorage
+    mGroupsProvider.SetStorageDelegate(&mDeviceStorage);
     err = mGroupsProvider.Init();
     SuccessOrExit(err);
     SetGroupDataProvider(&mGroupsProvider);

diff --git a/src/credentials/GroupDataProvider.h b/src/credentials/GroupDataProvider.h
@@ -221,8 +221,9 @@ class GroupDataProvider
     uint16_t GetMaxGroupKeysPerFabric() { return mMaxGroupKeysPerFabric; }
 
     /**
-     *  Initialize the GroupDataProvider, including any persistent data store
-     *  initialization. Must be called once before any other API succeeds.
+     *  Initialize the GroupDataProvider, including possibly any persistent
+     *  data store initialization done by the implementation. Must be called once
+     *  before any other API succeeds.
      *
      *  @retval #CHIP_ERROR_INCORRECT_STATE if called when already initialized.
      *  @retval #CHIP_NO_ERROR on success
-Original file line number
+Diff line change
@@ Expand Up @@
     }
     void InteractionModelEngine::CloseTransactionsFromFabricIndex(FabricIndex aFabricIndex)
-    {
+    {
         //
         // Walk through all existing subscriptions and shut down those whose subscriber matches
         // that which just came in.
@@ Expand Down @@