Added comment to warn about fabric table access

src/app/clusters/scenes-server/SceneTable.h

@@ -284,6 +284,14 @@ class SceneTable
     virtual CHIP_ERROR SceneApplyEFS(const SceneTableEntry & scene) = 0;
 
     // Fabrics
+
+    /**
+     * @brief Removes all scenes associated with a fabric index and the stored FabricSceneData that maps them
+     * @param fabric_index Fabric index to remove
+     * @return CHIP_ERROR, CHIP_NO_ERROR if successful or if the Fabric was not found, specific CHIP_ERROR otherwise
+     * @note This function is meant to be used after a fabric is removed from the device, the implementation MUST ensure that it
+     * won't interact with the actual fabric table as it will be removed beforehand.
+     */
     virtual CHIP_ERROR RemoveFabric(FabricIndex fabric_index) = 0;
     virtual CHIP_ERROR RemoveEndpoint()                       = 0;
 

src/app/clusters/scenes-server/scenes-server.cpp

@@ -110,6 +110,7 @@ class ScenesClusterFabricDelegate : public chip::FabricTable::Delegate
     {
         SceneTable * sceneTable = scenes::GetSceneTableImpl();
         VerifyOrReturn(nullptr != sceneTable);
+        // The implementation of SceneTable::RemoveFabric() must not call back into the FabricTable
         sceneTable->RemoveFabric(fabricIndex);
     }
 };