Editorial fixes

Author: nivi-apple

src/darwin/Framework/CHIP/MTROTAImageTransferHandler.mm

@@ -42,6 +42,8 @@
     Messaging::ExchangeContext * exchangeCtx, FabricIndex fabricIndex, NodeId nodeId)
 {
     assertChipStackLockedByCurrentThread();
+    VerifyOrReturnError(layer != nullptr, CHIP_ERROR_INCORRECT_STATE);
+
     mSystemLayer = layer;
 
     ReturnErrorOnFailure(ConfigureState(fabricIndex, nodeId));
@@ -279,7 +281,6 @@
     case TransferSession::OutputEventType::kAckEOFReceived:
     case TransferSession::OutputEventType::kInternalError:
     case TransferSession::OutputEventType::kTransferTimeout:
-        ChipLogError(BDX, "Got kAckEOFReceived/kInternalError/kTransferTimeout");
         err = OnTransferSessionEnd(event);
         if (err != CHIP_NO_ERROR) {
             LogErrorOnFailure(err);
@@ -343,7 +344,8 @@
     VerifyOrReturnError(ec != nullptr, CHIP_ERROR_INCORRECT_STATE);
     CHIP_ERROR err;
 
-    // If we receive a ReceiveInit message, then we prepare for transfer.
+    // If we receive a ReceiveInit message, then we prepare for transfer. Otherwise we send the message
+    // received to the AsyncTransferFacilitator for processing.
     if (payloadHeader.HasMessageType(MessageType::ReceiveInit)) {
         NodeId nodeId = ec->GetSessionHandle()->GetPeer().GetNodeId();
         FabricIndex fabricIndex = ec->GetSessionHandle()->GetFabricIndex();

src/darwin/Framework/CHIP/MTROTAProviderDelegateBridge.mm

@@ -244,7 +244,6 @@ bool GetPeerNodeInfo(CommandHandler * commandHandler, const ConcreteCommandPath
 
                 // If update is not available, return the delegate response
                 if (!hasUpdate) {
-                    ChipLogError(BDX, "delegate bridge. update not available");
                     handler->AddResponse(cachedCommandPath, delegateResponse);
                     handle.Release();
                     return;