From 12389f1a525f7bcc33ab0be81cfb3af3ef804414 Mon Sep 17 00:00:00 2001 From: Boris Zbarsky Date: Mon, 3 Oct 2022 12:10:40 -0400 Subject: [PATCH] Pass the right constant to ArmFailSafe. It takes a fabric index, not a fabric id. This happened to work because both kUndefinedFabricIndex and kUndefinedFabricId have the numeric value 0. --- src/app/server/CommissioningWindowManager.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/app/server/CommissioningWindowManager.cpp b/src/app/server/CommissioningWindowManager.cpp index e9f3ae6350d432..17503478b862bc 100644 --- a/src/app/server/CommissioningWindowManager.cpp +++ b/src/app/server/CommissioningWindowManager.cpp @@ -175,7 +175,7 @@ void CommissioningWindowManager::OnSessionEstablished(const SessionHandle & sess } else { - err = failSafeContext.ArmFailSafe(kUndefinedFabricId, System::Clock::Seconds16(60)); + err = failSafeContext.ArmFailSafe(kUndefinedFabricIndex, System::Clock::Seconds16(60)); if (err != CHIP_NO_ERROR) { ChipLogError(AppServer, "Error arming failsafe on PASE session establishment completion");