Fix svace defects for 1.2-rel (cloudResource.c)
authorAndrii Shtompel <a.shtompel@samsung.com>
Fri, 4 Nov 2016 10:39:53 +0000 (12:39 +0200)
committerRandeep Singh <randeep.s@samsung.com>
Tue, 8 Nov 2016 08:57:53 +0000 (08:57 +0000)
Change-Id: I74a65a64347e0832d8b42ec0fdf3ac0eab50a9cc
Signed-off-by: Andrii Shtompel <a.shtompel@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/14041
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Kevin Kane <kkane@microsoft.com>
Reviewed-by: Jongsung Lee <js126.lee@samsung.com>
Reviewed-by: Randeep Singh <randeep.s@samsung.com>
(cherry picked from commit 053384162d114770dd79e0a04da3898895cc9f29)
Reviewed-on: https://gerrit.iotivity.org/gerrit/14101

resource/csdk/security/provisioning/sample/cloud/cloudResource.c

index 174fdeb..bf43744 100644 (file)
@@ -93,14 +93,14 @@ static OCEntityHandlerResult ProcessRequest (OCEntityHandlerRequest *request,
 
     *payload = getPayload(gResourceUri, currLEDResource->power, currLEDResource->state);
 
-    return OC_EH_OK;
+    return (*payload)? OC_EH_OK : OC_EH_ERROR;
 }
 
 static OCEntityHandlerResult ProcessPostRequest (OCEntityHandlerRequest *request,
         char *resourceUri, OCRepPayload **payload)
 {
     OCRepPayload *answer = NULL;
-    OCEntityHandlerResult result = OC_EH_OK;
+    OCEntityHandlerResult result = OC_EH_ERROR;
 
     /*
      * The entity handler determines how to process a POST request.
@@ -142,7 +142,7 @@ static OCEntityHandlerResult ProcessPostRequest (OCEntityHandlerRequest *request
                 gLedInstance[gCurrLedInstance].power = 0;
                 gCurrLedInstance++;
                 strncpy(resourceUri, uri, MAX_URI_LENGTH);
-                result = OC_EH_RESOURCE_CREATED;
+                result = OC_EH_OK;
             }
         }
         else
@@ -162,15 +162,9 @@ static OCEntityHandlerResult ProcessPostRequest (OCEntityHandlerRequest *request
         }
     }
 
-    if (answer)
+    if (OC_EH_OK == result)
     {
         *payload = answer;
-        result = OC_EH_OK;
-    }
-    else
-    {
-        OIC_LOG_V (INFO, TAG, "Payload was NULL");
-        result = OC_EH_ERROR;
     }
 
     return result;