[IOT-1473] Modify the return value of MOT related API.
authorChul Lee <chuls.lee@samsung.com>
Wed, 26 Oct 2016 05:46:15 +0000 (14:46 +0900)
committerRandeep Singh <randeep.s@samsung.com>
Thu, 27 Oct 2016 10:51:07 +0000 (10:51 +0000)
Change-Id: I9e2a6d79358538896c64e5870d96c0ca796a0b6e
Signed-off-by: Chul Lee <chuls.lee@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/13687
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: dongik Lee <dongik.lee@samsung.com>
Reviewed-by: Randeep Singh <randeep.s@samsung.com>
resource/csdk/security/provisioning/src/multipleownershiptransfermanager.c

index 76c2684..492c184 100644 (file)
@@ -327,15 +327,16 @@ exit:
 OCStackResult MOTSelectMOTMethod(void *ctx, const OCProvisionDev_t *targetDeviceInfo,
                                  const OicSecOxm_t oxmSelValue, OCProvisionResultCB resultCallback)
 {
-    OCStackResult postMomRes = OC_STACK_INVALID_PARAM;
+    OCStackResult postMomRes = OC_STACK_INVALID_CALLBACK;
     OicSecDoxm_t* doxm = NULL;
     uint8_t* doxmPayload = NULL;
     size_t doxmPayloadLen = 0;
 
     OIC_LOG(DEBUG, TAG, "IN MOTSelectOTMethod");
 
-    VERIFY_NON_NULL(TAG, targetDeviceInfo, ERROR);
     VERIFY_NON_NULL(TAG, resultCallback, ERROR);
+    postMomRes = OC_STACK_INVALID_PARAM;
+    VERIFY_NON_NULL(TAG, targetDeviceInfo, ERROR);
 
     bool isValidOxmsel = false;
     for(size_t i = 0; i < targetDeviceInfo->doxm->oxmLen; i++)
@@ -383,7 +384,7 @@ exit:
 OCStackResult MOTProvisionPreconfigPIN(void *ctx, const OCProvisionDev_t *targetDeviceInfo,
                                  const char* preconfPIN, size_t preconfPINLen, OCProvisionResultCB resultCallback)
 {
-    OCStackResult postCredRes = OC_STACK_INVALID_PARAM;
+    OCStackResult postCredRes = OC_STACK_INVALID_CALLBACK;
     bool freeFlag = true;
     OCSecurityPayload* secPayload = NULL;
     MOTContext_t *motCtx = NULL;
@@ -391,8 +392,9 @@ OCStackResult MOTProvisionPreconfigPIN(void *ctx, const OCProvisionDev_t *target
 
     OIC_LOG(DEBUG, TAG, "IN MOTProvisionPreconfigPIN");
 
-    VERIFY_NON_NULL(TAG, targetDeviceInfo, ERROR);
     VERIFY_NON_NULL(TAG, resultCallback, ERROR);
+    postCredRes = OC_STACK_INVALID_PARAM;
+    VERIFY_NON_NULL(TAG, targetDeviceInfo, ERROR);
     VERIFY_NON_NULL(TAG, preconfPIN, ERROR);
     VERIFY_SUCCESS(TAG, (0 != preconfPINLen), ERROR);
     VERIFY_SUCCESS(TAG, (0 != preconfPINLen && OXM_PRECONFIG_PIN_SIZE >= preconfPINLen), ERROR);