From 7c28d9c76629039fbaf917d82534600cc1d443cd Mon Sep 17 00:00:00 2001 From: Nivedita Sarkar Date: Thu, 6 Jun 2024 17:16:42 -0700 Subject: [PATCH] Timeout for BDX log should be treated as an abort transfer on failure use case --- .../diagnostic-logs-server/BDXDiagnosticLogsProvider.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/app/clusters/diagnostic-logs-server/BDXDiagnosticLogsProvider.cpp b/src/app/clusters/diagnostic-logs-server/BDXDiagnosticLogsProvider.cpp index 6119652990679f..24293603544e37 100644 --- a/src/app/clusters/diagnostic-logs-server/BDXDiagnosticLogsProvider.cpp +++ b/src/app/clusters/diagnostic-logs-server/BDXDiagnosticLogsProvider.cpp @@ -230,14 +230,14 @@ void BDXDiagnosticLogsProvider::OnInternalError() { ChipLogError(BDX, "Internal Error"); VerifyOrDo(mIsAcceptReceived, SendCommandResponse(StatusEnum::kDenied)); - Reset(); + mTransfer.AbortTransfer(GetBdxStatusCodeFromChipError(CHIP_ERROR_INTERNAL)); } void BDXDiagnosticLogsProvider::OnTimeout() { ChipLogError(BDX, "Timeout"); VerifyOrDo(mIsAcceptReceived, SendCommandResponse(StatusEnum::kDenied)); - Reset(); + mTransfer.AbortTransfer(GetBdxStatusCodeFromChipError(CHIP_ERROR_TIMEOUT)); } void BDXDiagnosticLogsProvider::SendCommandResponse(StatusEnum status)