Fix MinInterval handling in ReadClient. (#19854)

We had codepaths on which we did not store MinInterval for an outgoing subscribe request, and would later report the wrong MinInterval.
project-chip · Nov 1, 2023 · 8836705 · 8836705
1 parent 16b5ba3
commit 8836705
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/app/ReadClient.cpp b/src/app/ReadClient.cpp
@@ -865,14 +865,15 @@ CHIP_ERROR ReadClient::SendSubscribeRequest(const ReadPrepareParams & aReadPrepa
 {
     VerifyOrReturnError(aReadPrepareParams.mMinIntervalFloorSeconds <= aReadPrepareParams.mMaxIntervalCeilingSeconds,
                         CHIP_ERROR_INVALID_ARGUMENT);
-    mMinIntervalFloorSeconds = aReadPrepareParams.mMinIntervalFloorSeconds;
     return SendSubscribeRequestImpl(aReadPrepareParams);
 }
 
 CHIP_ERROR ReadClient::SendSubscribeRequestImpl(const ReadPrepareParams & aReadPrepareParams)
 {
     VerifyOrReturnError(ClientState::Idle == mState, CHIP_ERROR_INCORRECT_STATE);
 
+    mMinIntervalFloorSeconds = aReadPrepareParams.mMinIntervalFloorSeconds;
+
     // Todo: Remove the below, Update span in ReadPrepareParams
     Span<AttributePathParams> attributePaths(aReadPrepareParams.mpAttributePathParamsList,
                                              aReadPrepareParams.mAttributePathParamsListSize);