From 053384162d114770dd79e0a04da3898895cc9f29 Mon Sep 17 00:00:00 2001 From: Andrii Shtompel Date: Fri, 4 Nov 2016 12:39:53 +0200 Subject: [PATCH] Fix svace defects for 1.2-rel (cloudResource.c) Change-Id: I74a65a64347e0832d8b42ec0fdf3ac0eab50a9cc Signed-off-by: Andrii Shtompel Reviewed-on: https://gerrit.iotivity.org/gerrit/14041 Tested-by: jenkins-iotivity Reviewed-by: Kevin Kane Reviewed-by: Jongsung Lee Reviewed-by: Randeep Singh --- .../security/provisioning/sample/cloud/cloudResource.c | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/resource/csdk/security/provisioning/sample/cloud/cloudResource.c b/resource/csdk/security/provisioning/sample/cloud/cloudResource.c index 174fdeb..bf43744 100644 --- a/resource/csdk/security/provisioning/sample/cloud/cloudResource.c +++ b/resource/csdk/security/provisioning/sample/cloud/cloudResource.c @@ -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; -- 2.7.4