Apply code review comment

project-chip · Dec 10, 2020 · bde1051 · bde1051
1 parent aad5122
commit bde1051
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/controller/CHIPDevice.cpp b/src/controller/CHIPDevice.cpp
@@ -268,9 +268,10 @@ CHIP_ERROR Device::LoadSecureSessionParameters(ResetTransport resetNeeded)
 
 bool Device::GetIpAddress(Inet::IPAddress & addr) const
 {
-    if (mState != ConnectionState::NotConnected)
-        addr = mDeviceAddr;
-    return mState != ConnectionState::NotConnected;
+    if (mState == ConnectionState::NotConnected)
+        return false;
+    addr = mDeviceAddr;
+    return true;
 }
 
 void Device::AddResponseHandler(EndpointId endpoint, ClusterId cluster, Callback::Callback<> * onResponse)