commit 7422693818ade8a574131436c8331ee05786bcd8
author yunhanw <yunhanw@google.com> Fri Feb 07 17:31:27 2020 -0800
committer yunhanw <yunhanw@google.com> Fri Feb 07 17:31:27 2020 -0800
tree f5fc50d8be9e105365bdf8e9ca9bd6f892b62859
parent 9708816ed536ce18e2a92824c47fb22bf57f97b3

update error value for WeaveDataManagementClient

src/device-manager/WeaveDataManagementClient.cpp

diff --git a/src/device-manager/WeaveDataManagementClient.cpp b/src/device-manager/WeaveDataManagementClient.cpp
index 6c7a55c..f5a940b 100644
--- a/src/device-manager/WeaveDataManagementClient.cpp
+++ b/src/device-manager/WeaveDataManagementClient.cpp
@@ -500,7 +500,7 @@
     SuccessOrExit(err);
 
     it = mPathTlvDataMap.find(propertyPathHandle);
-    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INCORRECT_STATE);
+    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INVALID_TLV_TAG);
 
     pMsgBuf = mPathTlvDataMap[propertyPathHandle];
 
@@ -539,7 +539,7 @@
     SuccessOrExit(err);
 
     it = mPathTlvDataMap.find(propertyPathHandle);
-    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INCORRECT_STATE);
+    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INVALID_TLV_TAG);
 
     pMsgBuf = mPathTlvDataMap[propertyPathHandle];
 
@@ -572,7 +572,7 @@
     PacketBuffer * pMsgBuf = PacketBuffer::New();
     VerifyOrExit(NULL != pMsgBuf, err = WEAVE_ERROR_NO_MEMORY);
 
-    VerifyOrExit(NULL != apBytesData, err = WEAVE_ERROR_INCORRECT_STATE);
+    VerifyOrExit(NULL != apBytesData, err = WEAVE_ERROR_INVALID_ARGUMENT);
 
     err = GetSchemaEngine()->MapPathToHandle(apPath, propertyPathHandle);
     SuccessOrExit(err);
@@ -620,7 +620,7 @@
     SuccessOrExit(err);
 
     it = mPathTlvDataMap.find(propertyPathHandle);
-    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INCORRECT_STATE);
+    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INVALID_TLV_TAG);
 
     pMsgBuf = mPathTlvDataMap[propertyPathHandle];
 
@@ -661,7 +661,7 @@
     SuccessOrExit(err);
 
     it = mPathTlvDataMap.find(propertyPathHandle);
-    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INCORRECT_STATE);
+    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INVALID_TLV_TAG);
 
     pMsgBuf = mPathTlvDataMap[propertyPathHandle];
 
@@ -714,7 +714,7 @@
     SuccessOrExit(err);
 
     it = mPathTlvDataMap.find(propertyPathHandle);
-    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INCORRECT_STATE);
+    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INVALID_TLV_TAG);
 
     pMsgBuf = mPathTlvDataMap[propertyPathHandle];
 
@@ -779,7 +779,7 @@
 
     std::map<PropertyPathHandle, PacketBuffer *>::iterator it = mPathTlvDataMap.find(aLeafHandle);
 
-    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INCORRECT_STATE);
+    VerifyOrExit(it != mPathTlvDataMap.end(), err = WEAVE_ERROR_INVALID_TLV_TAG);
 
     pMsgBuf = mPathTlvDataMap[aLeafHandle];
 
@@ -927,7 +927,7 @@
             else
             {
                 traitListLen = pWdmClient->mSinkCatalog.Size();
-                VerifyOrExit(traitListLen != 0, err = WEAVE_ERROR_INCORRECT_STATE);
+                VerifyOrExit(traitListLen != 0, err = WEAVE_ERROR_INVALID_LIST_LENGTH);
             }
             WeaveLogDetail(DataManagement, "prepare to subscribe %d trait data sink", traitListLen);