From 19e2b58cbee8ea4314552de17b67d0831fd5e5b5 Mon Sep 17 00:00:00 2001 From: "jihwan.seo" Date: Fri, 9 Sep 2016 18:26:07 +0900 Subject: [PATCH] remove warning for keepalive - check return value for OCDoresource Change-Id: Ia3efa0d8365dd60ecb0c0e7dc751d242bcffa97d Signed-off-by: jihwan.seo Reviewed-on: https://gerrit.iotivity.org/gerrit/11633 Tested-by: jenkins-iotivity Reviewed-by: Jaehong Jo Reviewed-by: Hyuna Jo Reviewed-by: Ashok Babu Channa --- resource/csdk/stack/src/oickeepalive.c | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/resource/csdk/stack/src/oickeepalive.c b/resource/csdk/stack/src/oickeepalive.c index 8acd7d0..aabb391 100644 --- a/resource/csdk/stack/src/oickeepalive.c +++ b/resource/csdk/stack/src/oickeepalive.c @@ -579,8 +579,14 @@ OCStackResult SendPingMessage(KeepAliveEntry_t *entry) payload->base.type = PAYLOAD_TYPE_REPRESENTATION; OCRepPayloadSetPropInt(payload, INTERVAL, entry->interval); - OCDoResource(NULL, OC_REST_PUT, KEEPALIVE_RESOURCE_URI, &devAddr, - (OCPayload *) payload, CT_ADAPTER_TCP, OC_LOW_QOS, &pingData, NULL, 0); + OCStackResult result = OCDoResource(NULL, OC_REST_PUT, KEEPALIVE_RESOURCE_URI, &devAddr, + (OCPayload *) payload, CT_ADAPTER_TCP, OC_LOW_QOS, + &pingData, NULL, 0); + if (OC_STACK_OK != result) + { + OIC_LOG(ERROR, TAG, "OCDoResource has failed"); + return result; + } // Update timeStamp with time sent ping message for next ping message. entry->timeStamp = OICGetCurrentTime(TIME_IN_US); @@ -741,8 +747,14 @@ void HandleKeepAliveConnCB(const CAEndpoint_t *endpoint, bool isConnected) OCDevAddr devAddr = { .adapter = OC_ADAPTER_TCP }; CopyEndpointToDevAddr(endpoint, &devAddr); - OCDoResource(NULL, OC_REST_DISCOVER, KEEPALIVE_RESOURCE_URI, &devAddr, NULL, - CT_ADAPTER_TCP, OC_HIGH_QOS, &pingData, NULL, 0); + OCStackResult result = OCDoResource(NULL, OC_REST_DISCOVER, KEEPALIVE_RESOURCE_URI, + &devAddr, NULL, CT_ADAPTER_TCP, OC_HIGH_QOS, + &pingData, NULL, 0); + if (OC_STACK_OK != result) + { + OIC_LOG(ERROR, TAG, "OCDoResource has failed"); + return; + } } else { -- 2.7.4