[SetUpCodePairer] Only search on network if no credentials are provid…

…ed to chip-tool pairing code commands (#22104) (#22166) Co-authored-by: Vivien Nicolas <[email protected]>
project-chip · Aug 29, 2022 · 0e5b2dd · 0e5b2dd
1 parent 17c5878
commit 0e5b2dd
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 12 deletions.
diff --git a/src/controller/CHIPDeviceController.cpp b/src/controller/CHIPDeviceController.cpp
@@ -571,7 +571,16 @@ CHIP_ERROR DeviceCommissioner::PairDevice(NodeId remoteDeviceId, const char * se
         return CHIP_ERROR_INCORRECT_STATE;
     }
     ReturnErrorOnFailure(mDefaultCommissioner->SetCommissioningParameters(params));
-    return mSetUpCodePairer.PairDevice(remoteDeviceId, setUpCode, SetupCodePairerBehaviour::kCommission);
+
+    auto threadCredentials = params.GetThreadOperationalDataset();
+    auto wiFiCredentials   = params.GetWiFiCredentials();
+    bool hasCredentials    = threadCredentials.HasValue() || wiFiCredentials.HasValue();
+
+    // If there is no network credentials, we assume that the pairing command as been issued to pair with a
+    // device that is already on the network.
+    auto pairerBehaviour = hasCredentials ? SetupCodePairerBehaviour::kCommission : SetupCodePairerBehaviour::kCommissionOnNetwork;
+
+    return mSetUpCodePairer.PairDevice(remoteDeviceId, setUpCode, pairerBehaviour);
 }
 
 CHIP_ERROR DeviceCommissioner::PairDevice(NodeId remoteDeviceId, const char * setUpCode)

diff --git a/src/controller/SetUpCodePairer.cpp b/src/controller/SetUpCodePairer.cpp
@@ -72,22 +72,26 @@ CHIP_ERROR SetUpCodePairer::Connect(SetupPayload & payload)
     bool isRunning = false;
 
     bool searchOverAll = !payload.rendezvousInformation.HasValue();
-    if (searchOverAll || payload.rendezvousInformation.Value().Has(RendezvousInformationFlag::kBLE))
+
+    if (mConnectionType != SetupCodePairerBehaviour::kCommissionOnNetwork)
     {
-        if (CHIP_NO_ERROR == (err = StartDiscoverOverBle(payload)))
+        if (searchOverAll || payload.rendezvousInformation.Value().Has(RendezvousInformationFlag::kBLE))
         {
-            isRunning = true;
+            if (CHIP_NO_ERROR == (err = StartDiscoverOverBle(payload)))
+            {
+                isRunning = true;
+            }
+            VerifyOrReturnError(searchOverAll || CHIP_NO_ERROR == err || CHIP_ERROR_UNSUPPORTED_CHIP_FEATURE == err, err);
         }
-        VerifyOrReturnError(searchOverAll || CHIP_NO_ERROR == err || CHIP_ERROR_UNSUPPORTED_CHIP_FEATURE == err, err);
-    }
 
-    if (searchOverAll || payload.rendezvousInformation.Value().Has(RendezvousInformationFlag::kSoftAP))
-    {
-        if (CHIP_NO_ERROR == (err = StartDiscoverOverSoftAP(payload)))
+        if (searchOverAll || payload.rendezvousInformation.Value().Has(RendezvousInformationFlag::kSoftAP))
         {
-            isRunning = true;
+            if (CHIP_NO_ERROR == (err = StartDiscoverOverSoftAP(payload)))
+            {
+                isRunning = true;
+            }
+            VerifyOrReturnError(searchOverAll || CHIP_NO_ERROR == err || CHIP_ERROR_UNSUPPORTED_CHIP_FEATURE == err, err);
         }
-        VerifyOrReturnError(searchOverAll || CHIP_NO_ERROR == err || CHIP_ERROR_UNSUPPORTED_CHIP_FEATURE == err, err);
     }
 
     // We always want to search on network because any node that has already been commissioned will use on-network regardless of the
@@ -225,7 +229,8 @@ bool SetUpCodePairer::ConnectToDiscoveredDevice()
         ExpectPASEEstablishment();
 
         CHIP_ERROR err;
-        if (mConnectionType == SetupCodePairerBehaviour::kCommission)
+        if (mConnectionType == SetupCodePairerBehaviour::kCommission ||
+            mConnectionType == SetupCodePairerBehaviour::kCommissionOnNetwork)
         {
             err = mCommissioner->PairDevice(mRemoteId, params);
         }

diff --git a/src/controller/SetUpCodePairer.h b/src/controller/SetUpCodePairer.h
@@ -51,6 +51,7 @@ class DeviceCommissioner;
 enum class SetupCodePairerBehaviour : uint8_t
 {
     kCommission,
+    kCommissionOnNetwork,
     kPaseOnly,
 };