Deprecate resourceHandle parameter in OCEntityHandlerResponse 17/20517/8
authorJaewook Jung <jw0213.jung@samsung.com>
Thu, 1 Jun 2017 13:12:49 +0000 (22:12 +0900)
committerDan Mihai <Daniel.Mihai@microsoft.com>
Wed, 14 Jun 2017 15:39:44 +0000 (15:39 +0000)
resourceHandle of OCEntityHandlerResponse is not used.

Change-Id: I7630e0cd891872a7788873ecd75a738b8e0c362d
Signed-off-by: Jaewook Jung <jw0213.jung@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/20517
Tested-by: jenkins-iotivity <jenkins@iotivity.org>
Reviewed-by: Dan Mihai <Daniel.Mihai@microsoft.com>
35 files changed:
bridging/common/ConcurrentIotivityUtils.cpp
resource/csdk/include/octypes.h
resource/csdk/resource-directory/src/rd_server.c
resource/csdk/routing/include/routingmanager.h
resource/csdk/routing/include/routingmanagerinterface.h
resource/csdk/routing/src/routingmanager.c
resource/csdk/routing/src/routingmanagerinterface.c
resource/csdk/security/provisioning/sample/cloud/cloudResource.c
resource/csdk/security/provisioning/sample/sampleserver_justworks.cpp
resource/csdk/security/provisioning/sample/sampleserver_mfg.cpp
resource/csdk/security/provisioning/sample/sampleserver_mvjustworks.cpp
resource/csdk/security/provisioning/sample/sampleserver_preconfpin.cpp
resource/csdk/security/provisioning/sample/sampleserver_randompin.cpp
resource/csdk/security/provisioning/unittest/sampleserver1.cpp
resource/csdk/security/provisioning/unittest/sampleserver2.cpp
resource/csdk/security/src/resourcemanager.c
resource/csdk/stack/samples/arduino/SimpleClientServer/ocserver/ocserver.cpp
resource/csdk/stack/samples/linux/SimpleClientServer/ocserver.cpp
resource/csdk/stack/samples/linux/SimpleClientServer/ocserverbasicops.cpp
resource/csdk/stack/samples/linux/SimpleClientServer/ocservercoll.cpp
resource/csdk/stack/samples/linux/SimpleClientServer/ocserverslow.cpp
resource/csdk/stack/samples/linux/secure/ocserverbasicops.cpp
resource/csdk/stack/samples/tizen/SimpleClientServer/ocserver.cpp
resource/csdk/stack/src/occollection.c
resource/csdk/stack/src/ocobserve.c
resource/csdk/stack/src/ocresource.c
resource/csdk/stack/src/oicgroup.c
resource/csdk/stack/src/oickeepalive.c
resource/csdk/stack/test/stacktests.cpp
resource/src/InProcServerWrapper.cpp
service/coap-http-proxy/src/CoapHttpHandler.c
service/easy-setup/enrollee/src/resourcehandler.c
service/notification/src/provider/NSProviderListener.c
service/notification/src/provider/NSProviderSubscription.c
service/notification/src/provider/NSProviderTopic.c

index 0c77e4b..3c23fa4 100644 (file)
@@ -72,7 +72,6 @@ OCStackResult ConcurrentIotivityUtils::respondToRequest(OCEntityHandlerRequest *
     std::unique_ptr<OCEntityHandlerResponse> response = make_unique<OCEntityHandlerResponse>();
 
     response->requestHandle = request->requestHandle;
-    response->resourceHandle = request->resource;
     response->ehResult = responseCode;
 
     // Clone a copy since this allocation is going across thread boundaries.
index 7234d78..19abc2d 100644 (file)
@@ -1699,7 +1699,7 @@ typedef struct
     /** Request handle.*/
     OCRequestHandle requestHandle;
 
-    /** Resource handle.*/
+    /** Resource handle. (@deprecated: This parameter is not used.) */
     OCResourceHandle resourceHandle;
 
     /** Allow the entity handler to pass a result with the response.*/
index d1f0c25..3c9bb93 100644 (file)
@@ -44,7 +44,6 @@ static OCStackResult sendResponse(const OCEntityHandlerRequest *ehRequest, OCRep
 {
     OCEntityHandlerResponse response = { 0 };
     response.requestHandle = ehRequest->requestHandle;
-    response.resourceHandle = ehRequest->resource;
     response.ehResult = ehResult;
     response.payload = (OCPayload*)(rdPayload);
     return OCDoResponse(&response);
index 088fdd0..8f797d5 100644 (file)
@@ -52,10 +52,9 @@ OCStackResult RMTerminate();
  * This API will be called from RI layer whenever there is a request for the GATEWAY
  * Virtual Resource.
  * @param[in,out]   request    Request Received.
- * @param[in]       resource   Resource handle used for sending the response.
  * @return  ::OC_STACK_OK or Appropriate error code.
  */
-OCStackResult RMHandleGatewayRequest(OCServerRequest *request, const OCResource *resource);
+OCStackResult RMHandleGatewayRequest(OCServerRequest *request);
 
 /**
  * API to handle the Response payload.  The Gateway entries are parsed from
index 8563b5b..9a012bf 100644 (file)
@@ -65,12 +65,10 @@ OCStackResult RMSendDeleteRequest(const OCDevAddr *devAddr, OCRepPayload *payloa
 /**
  * Send a response for GET/OBSERVE request received for Gateway resource.
  * @param[in]   request     Request Received.
- * @param[in]   resource    Resource Handle.
  * @param[in]   payload     Payload containing Gateway Entries.
  * @return  ::OC_STACK_OK or Appropriate error code.
  */
-OCStackResult RMSendResponse(const OCServerRequest *request, const OCResource *resource,
-                             const OCRepPayload *payload);
+OCStackResult RMSendResponse(const OCServerRequest *request, const OCRepPayload *payload);
 
 /**
  * Send notification for list of observers except a particular observer.
index bf16189..6210f98 100644 (file)
@@ -96,26 +96,23 @@ static bool g_isRMInitialized = false;
 /**
  * API to handle the GET request received for a Gateway Resource.
  * @param[in]   request     Request Received.
- * @param[in]   resource    Resource handle used for sending the response.
  * @return  ::OC_STACK_OK or Appropriate error code.
  */
-OCStackResult RMHandleGETRequest(const OCServerRequest *request, const OCResource *resource);
+OCStackResult RMHandleGETRequest(const OCServerRequest *request);
 
 /**
  * API to handle the OBSERVE request received for a Gateway Resource.
  * @param[in,out]   request    Request Received.
- * @param[in]       resource   Resource handle used for sending the response.
  * @return  ::OC_STACK_OK or Appropriate error code.
  */
-OCStackResult RMHandleOBSERVERequest(OCServerRequest *request, const OCResource *resource);
+OCStackResult RMHandleOBSERVERequest(OCServerRequest *request);
 
 /**
  * API to handle the OBSERVE request received for a Gateway Resource.
  * @param[in,out]   request    Request Received.
- * @param[in]       resource   Resource handle used for sending the response.
  * @return  ::OC_STACK_OK or Appropriate error code.
  */
-OCStackResult RMHandleDELETERequest(const OCServerRequest *request, const OCResource *resource);
+OCStackResult RMHandleDELETERequest(const OCServerRequest *request);
 
 /**
  * Adds a observer after generating observer ID whenever observe
@@ -231,7 +228,7 @@ OCStackResult RMTerminate()
     return result;
 }
 
-OCStackResult RMHandleGatewayRequest(OCServerRequest *request, const OCResource *resource)
+OCStackResult RMHandleGatewayRequest(OCServerRequest *request)
 {
     OIC_LOG(DEBUG, TAG, "RMHandleGatewayRequest IN");
 
@@ -242,7 +239,6 @@ OCStackResult RMHandleGatewayRequest(OCServerRequest *request, const OCResource
     }
 
     RM_NULL_CHECK_WITH_RET(request, TAG, "request");
-    RM_NULL_CHECK_WITH_RET(resource, TAG, "resource");
 
     OIC_LOG_V(DEBUG, TAG, "Received request of method: %d", request->method);
 
@@ -252,7 +248,7 @@ OCStackResult RMHandleGatewayRequest(OCServerRequest *request, const OCResource
         {
             case OC_OBSERVE_REGISTER:
                 OIC_LOG(DEBUG, TAG, "Received OBSERVE request");
-                RMHandleOBSERVERequest(request, resource);
+                RMHandleOBSERVERequest(request);
                 break;
             case OC_OBSERVE_DEREGISTER:
                 //TODO: Handle this case
@@ -260,7 +256,7 @@ OCStackResult RMHandleGatewayRequest(OCServerRequest *request, const OCResource
                 break;
             case OC_OBSERVE_NO_OPTION:
                 OIC_LOG(DEBUG, TAG, "Received GET request");
-                RMHandleGETRequest(request, resource);
+                RMHandleGETRequest(request);
                 break;
             default:
                 OIC_LOG(DEBUG, TAG, "Not Supported by Routing Manager");
@@ -269,7 +265,7 @@ OCStackResult RMHandleGatewayRequest(OCServerRequest *request, const OCResource
     else if (OC_REST_DELETE == request->method)
     {
         OIC_LOG(DEBUG, TAG, "Received a Delete request");
-        RMHandleDELETERequest(request, resource);
+        RMHandleDELETERequest(request);
     }
     OIC_LOG(DEBUG, TAG, "RMHandleGatewayRequest OUT");
     return OC_STACK_OK;
@@ -549,11 +545,10 @@ exit:
     return OC_STACK_OK;
 }
 
-OCStackResult RMHandleGETRequest(const OCServerRequest *request, const OCResource *resource)
+OCStackResult RMHandleGETRequest(const OCServerRequest *request)
 {
     OIC_LOG(DEBUG, TAG, "RMHandleGETRequest IN");
     RM_NULL_CHECK_WITH_RET(request, TAG, "request");
-    RM_NULL_CHECK_WITH_RET(resource, TAG, "resource");
 
     OCRepPayload *payload = NULL;
     OCStackResult result = RMPConstructGatewayPayload(g_GatewayID, &payload);
@@ -564,7 +559,7 @@ OCStackResult RMHandleGETRequest(const OCServerRequest *request, const OCResourc
     }
 
     // Send a response for GET request
-    result = RMSendResponse(request, resource, payload);
+    result = RMSendResponse(request, payload);
     if (OC_STACK_OK != result)
     {
         OIC_LOG_V(DEBUG, TAG, "Send response failed[%d]", result);
@@ -583,11 +578,10 @@ OCStackResult RMHandleGETRequest(const OCServerRequest *request, const OCResourc
     return result;
 }
 
-OCStackResult RMHandleOBSERVERequest(OCServerRequest *request, const OCResource *resource)
+OCStackResult RMHandleOBSERVERequest(OCServerRequest *request)
 {
     OIC_LOG(DEBUG, TAG, "RMHandleOBSERVERequest IN");
     RM_NULL_CHECK_WITH_RET(request, TAG, "request");
-    RM_NULL_CHECK_WITH_RET(resource, TAG, "resource");
 
     // Parse payload and add the gateway entry.
     if (0 < request->payloadSize)
@@ -616,7 +610,7 @@ OCStackResult RMHandleOBSERVERequest(OCServerRequest *request, const OCResource
         goto exit;
     }
 
-    result = RMSendResponse(request, resource, payload);
+    result = RMSendResponse(request, payload);
     RMPFreePayload(payload);
     RM_VERIFY_SUCCESS(result, OC_STACK_OK);
 exit:
@@ -624,11 +618,10 @@ exit:
     return result;
 }
 
-OCStackResult RMHandleDELETERequest(const OCServerRequest *request, const OCResource *resource)
+OCStackResult RMHandleDELETERequest(const OCServerRequest *request)
 {
     OIC_LOG(DEBUG, TAG, "RMHandleDELETERequest IN");
     RM_NULL_CHECK_WITH_RET(request, TAG, "request");
-    RM_NULL_CHECK_WITH_RET(resource, TAG, "resource");
 
     uint32_t gatewayId = 0;
     OCStackResult result = RMPParseRequestPayload(request->payload, request->payloadSize,
index 922f73f..7264dd9 100644 (file)
@@ -189,15 +189,13 @@ OCStackResult RMSendDeleteRequest(const OCDevAddr *devAddr, OCRepPayload *payloa
                     &deleteCb, NULL, 0);
 }
 
-OCStackResult RMSendResponse(const OCServerRequest *request, const OCResource *resource,
-                             const OCRepPayload *payload)
+OCStackResult RMSendResponse(const OCServerRequest *request, const OCRepPayload *payload)
 {
     OIC_LOG(DEBUG, TAG, "RMSendResponse IN");
     OCEntityHandlerResponse response = {.ehResult = OC_EH_OK,
                                         .payload = (OCPayload *)payload,
                                         .persistentBufferFlag = 0,
-                                        .requestHandle = (OCRequestHandle) request,
-                                        .resourceHandle = (OCResourceHandle) resource
+                                        .requestHandle = (OCRequestHandle) request
                                         };
     OIC_LOG(DEBUG, TAG, "RMSendResponse OUT");
 
index 1481b5c..3817790 100644 (file)
@@ -228,7 +228,6 @@ OCLedHandler (OCEntityHandlerFlag flag,
             memset(&response, 0, sizeof(OCEntityHandlerResponse));
 
             response.requestHandle = request->requestHandle;
-            response.resourceHandle = request->resource;
             response.ehResult = result;
             response.payload = (OCPayload*)payload;
             response.numSendVendorSpecificHeaderOptions = 0;
index 20a0c4a..3a59922 100644 (file)
@@ -372,7 +372,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = (OCPayload*)(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index 586d237..2b6aaf7 100644 (file)
@@ -370,7 +370,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = (OCPayload*)(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index e35a1e6..4e1230b 100644 (file)
@@ -373,7 +373,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = (OCPayload*)(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index cb553c5..9a42ab0 100644 (file)
@@ -372,7 +372,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = (OCPayload*)(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index d135f32..e7fe1e5 100644 (file)
@@ -408,7 +408,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = (OCPayload*)(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index 34fe14d..dfb9143 100644 (file)
@@ -359,7 +359,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = (OCPayload*)(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index 596ed19..ad1c26c 100644 (file)
@@ -362,7 +362,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = (OCPayload*)(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index 9b48738..ac9b2c2 100644 (file)
@@ -57,7 +57,6 @@ OCStackResult SendSRMResponse(const OCEntityHandlerRequest *ehRequest,
         OCSecurityPayload ocPayload = {.base = {.type = PAYLOAD_TYPE_INVALID}};
 
         response.requestHandle = ehRequest->requestHandle;
-        response.resourceHandle = ehRequest->resource;
         response.ehResult = ehRet;
         response.payload = (OCPayload *)(&ocPayload);
         response.payload->type = PAYLOAD_TYPE_SECURITY;
index 55ed42c..4c07941 100644 (file)
@@ -181,7 +181,6 @@ OCEntityHandlerResult OCEntityHandlerCb(OCEntityHandlerFlag flag, OCEntityHandle
         {
             // Format the response.  Note this requires some info about the request
             response.requestHandle = entityHandlerRequest->requestHandle;
-            response.resourceHandle = entityHandlerRequest->resource;
             response.ehResult = ehRet;
             response.payload = (OCPayload*) payload;
             response.numSendVendorSpecificHeaderOptions = 0;
index 40aa69b..96b87f7 100644 (file)
@@ -517,7 +517,6 @@ OCDeviceEntityHandlerCb (OCEntityHandlerFlag flag,
         {
             // Format the response.  Note this requires some info about the request
             response.requestHandle = entityHandlerRequest->requestHandle;
-            response.resourceHandle = entityHandlerRequest->resource;
             response.ehResult = ehResult;
             response.payload = reinterpret_cast<OCPayload*>(payload);
             // Indicate that response is NOT in a persistent buffer
@@ -616,7 +615,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
         {
             // Format the response.  Note this requires some info about the request
             response.requestHandle = entityHandlerRequest->requestHandle;
-            response.resourceHandle = entityHandlerRequest->resource;
             response.ehResult = ehResult;
             response.payload = reinterpret_cast<OCPayload*>(payload);
             // Indicate that response is NOT in a persistent buffer
index 2af8e5e..9ca0de3 100644 (file)
@@ -275,7 +275,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = reinterpret_cast<OCPayload*>(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index bedce0d..3d87952 100644 (file)
@@ -143,7 +143,6 @@ OCDeviceEntityHandlerCb (OCEntityHandlerFlag flag,
     {
         // Format the response.  Note this requires some info about the request
         response.requestHandle = entityHandlerRequest->requestHandle;
-        response.resourceHandle = entityHandlerRequest->resource;
         response.ehResult = ehResult;
         response.payload = nullptr;
         response.numSendVendorSpecificHeaderOptions = 0;
@@ -225,7 +224,6 @@ OCEntityHandlerResult OCEntityHandlerRoomCb(OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = ehRequest->requestHandle;
-                response.resourceHandle = ehRequest->resource;
                 response.ehResult = ret;
                 response.payload = reinterpret_cast<OCPayload*>(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
@@ -298,7 +296,6 @@ OCEntityHandlerResult OCEntityHandlerRoomCb(OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = ehRequest->requestHandle;
-                response.resourceHandle = ehRequest->resource;
                 response.ehResult = ret;
                 response.payload = reinterpret_cast<OCPayload*>(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
@@ -365,7 +362,6 @@ OCEntityHandlerResult OCEntityHandlerLightCb(OCEntityHandlerFlag flag,
         {
             // Format the response.  Note this requires some info about the request
             response.requestHandle = ehRequest->requestHandle;
-            response.resourceHandle = ehRequest->resource;
             response.ehResult = ret;
             response.payload = reinterpret_cast<OCPayload*>(payload);
             response.numSendVendorSpecificHeaderOptions = 0;
@@ -428,7 +424,6 @@ OCEntityHandlerResult OCEntityHandlerFanCb(OCEntityHandlerFlag flag,
         {
             // Format the response.  Note this requires some info about the request
             response.requestHandle = ehRequest->requestHandle;
-            response.resourceHandle = ehRequest->resource;
             response.ehResult = ret;
             response.payload = reinterpret_cast<OCPayload*>(payload);
             response.numSendVendorSpecificHeaderOptions = 0;
index 47355ea..46e81fc 100644 (file)
@@ -132,7 +132,6 @@ void ProcessGetPutRequest (OCEntityHandlerRequest *ehRequest)
 
     // Format the response.  Note this requires some info about the request
     response.requestHandle = ehRequest->requestHandle;
-    response.resourceHandle = ehRequest->resource;
     response.ehResult = OC_EH_OK;
     response.payload = reinterpret_cast<OCPayload*> (getResp);
     response.numSendVendorSpecificHeaderOptions = 0;
index 6c2c360..1ea8a93 100644 (file)
@@ -282,7 +282,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
             {
                 // Format the response.  Note this requires some info about the request
                 response.requestHandle = entityHandlerRequest->requestHandle;
-                response.resourceHandle = entityHandlerRequest->resource;
                 response.ehResult = ehResult;
                 response.payload = reinterpret_cast<OCPayload*>(payload);
                 response.numSendVendorSpecificHeaderOptions = 0;
index fbb8fd6..097fc8b 100644 (file)
@@ -488,7 +488,6 @@ OCDeviceEntityHandlerCb (OCEntityHandlerFlag flag,
         {
             // Format the response.  Note this requires some info about the request
             response.requestHandle = entityHandlerRequest->requestHandle;
-            response.resourceHandle = entityHandlerRequest->resource;
             response.ehResult = ehResult;
             response.payload = reinterpret_cast<OCPayload*>(payload);
             // Indicate that response is NOT in a persistent buffer
@@ -590,7 +589,6 @@ OCEntityHandlerCb (OCEntityHandlerFlag flag,
         {
             // Format the response.  Note this requires some info about the request
             response.requestHandle = entityHandlerRequest->requestHandle;
-            response.resourceHandle = entityHandlerRequest->resource;
             response.ehResult = ehResult;
             response.payload = reinterpret_cast<OCPayload*>(payload);
             // Indicate that response is NOT in a persistent buffer
index 13d44d8..cf10823 100644 (file)
@@ -94,15 +94,14 @@ static bool AddRTSBaslinePayload(OCRepPayload **linkArray, int size, OCRepPayloa
     return b;
 }
 
-static OCStackResult SendResponse(const OCRepPayload *payload, const OCEntityHandlerRequest *ehRequest,
-    const OCResource* collResource, OCEntityHandlerResult ehResult)
+static OCStackResult SendResponse(const OCRepPayload *payload,
+                        const OCEntityHandlerRequest *ehRequest, OCEntityHandlerResult ehResult)
 {
     OCEntityHandlerResponse response = {0};
     response.ehResult = ehResult;
     response.payload = (OCPayload*)payload;
     response.persistentBufferFlag = 0;
     response.requestHandle = (OCRequestHandle) ehRequest->requestHandle;
-    response.resourceHandle = (OCResourceHandle) collResource;
     return OCDoResponse(&response);
 }
 
@@ -185,7 +184,7 @@ exit:
     {
         ehResult = (ret == OC_STACK_NO_RESOURCE) ? OC_EH_RESOURCE_NOT_FOUND : OC_EH_ERROR;
     }
-    ret = SendResponse(colPayload, ehRequest, collResource, ehResult);
+    ret = SendResponse(colPayload, ehRequest, ehResult);
     OIC_LOG_PAYLOAD(DEBUG, (OCPayload *)colPayload);
     OCRepPayloadDestroy(colPayload);
     return ret;
@@ -309,7 +308,7 @@ OCStackResult DefaultCollectionEntityHandler(OCEntityHandlerFlag flag, OCEntityH
 exit:
     if (result != OC_STACK_OK)
     {
-        result = SendResponse(NULL, ehRequest, (OCResource *)ehRequest->resource, OC_EH_BAD_REQ);
+        result = SendResponse(NULL, ehRequest, OC_EH_BAD_REQ);
     }
     OICFree(ifQueryParam);
     OICFree(rtQueryParam);
index b98425c..e61a6be 100644 (file)
@@ -201,7 +201,6 @@ OCStackResult SendAllObserverNotification (OCMethod method, OCResource *resPtr,
                         ehResponse.payload = (OCPayload*)presenceResBuf;
                         ehResponse.persistentBufferFlag = 0;
                         ehResponse.requestHandle = (OCRequestHandle) request;
-                        ehResponse.resourceHandle = (OCResourceHandle) resPtr;
                         OICStrcpy(ehResponse.resourceUri, sizeof(ehResponse.resourceUri),
                                 resourceObserver->resUri);
                         result = OCDoResponse(&ehResponse);
@@ -287,7 +286,6 @@ OCStackResult SendListObserverNotification (OCResource * resource,
                         memcpy(ehResponse.payload, payload, sizeof(*payload));
                         ehResponse.persistentBufferFlag = 0;
                         ehResponse.requestHandle = (OCRequestHandle) request;
-                        ehResponse.resourceHandle = (OCResourceHandle) resource;
                         result = OCDoResponse(&ehResponse);
                         if (result == OC_STACK_OK)
                         {
index bc98191..bd4b0ea 100644 (file)
@@ -1593,7 +1593,7 @@ static bool includeThisResourceInResponse(OCResource *resource,
            resourceMatchesRTFilter(resource, resourceTypeFilter);
 }
 
-OCStackResult SendNonPersistantDiscoveryResponse(OCServerRequest *request, OCResource *resource,
+static OCStackResult SendNonPersistantDiscoveryResponse(OCServerRequest *request,
                                 OCPayload *discoveryPayload, OCEntityHandlerResult ehResult)
 {
     OCEntityHandlerResponse *response = NULL;
@@ -1606,7 +1606,6 @@ OCStackResult SendNonPersistantDiscoveryResponse(OCServerRequest *request, OCRes
     response->payload = discoveryPayload;
     response->persistentBufferFlag = 0;
     response->requestHandle = (OCRequestHandle) request;
-    response->resourceHandle = (OCResourceHandle) resource;
 
     result = OCDoResponse(response);
 
@@ -2006,7 +2005,7 @@ static OCStackResult HandleVirtualResource (OCServerRequest *request, OCResource
     {
         // Received request for a gateway
         OIC_LOG(INFO, TAG, "Request is for Gateway Virtual Request");
-        discoveryResult = RMHandleGatewayRequest(request, resource);
+        discoveryResult = RMHandleGatewayRequest(request);
     }
 #endif
     else if (OC_INTROSPECTION_URI == virtualUriInRequest)
@@ -2066,7 +2065,7 @@ static OCStackResult HandleVirtualResource (OCServerRequest *request, OCResource
         OIC_LOG_PAYLOAD(DEBUG, payload);
         if(discoveryResult == OC_STACK_OK)
         {
-            SendNonPersistantDiscoveryResponse(request, resource, payload, OC_EH_OK);
+            SendNonPersistantDiscoveryResponse(request, payload, OC_EH_OK);
         }
         else // Error handling
         {
@@ -2074,7 +2073,7 @@ static OCStackResult HandleVirtualResource (OCServerRequest *request, OCResource
             {
                 OIC_LOG_V(ERROR, TAG, "Sending a (%d) error to (%d) discovery request",
                     discoveryResult, virtualUriInRequest);
-                SendNonPersistantDiscoveryResponse(request, resource, NULL,
+                SendNonPersistantDiscoveryResponse(request, NULL,
                     (discoveryResult == OC_STACK_NO_RESOURCE) ?
                         OC_EH_RESOURCE_NOT_FOUND : OC_EH_ERROR);
             }
index 836166c..d14a642 100644 (file)
@@ -889,7 +889,6 @@ OCStackApplicationResult ActionSetCB(void* context, OCDoHandle handle,
 
         // Format the response.  Note this requires some info about the request
         response.requestHandle = info->ehRequest;
-        response.resourceHandle = info->collResource;
         response.payload = clientResponse->payload;
         response.numSendVendorSpecificHeaderOptions = 0;
         memset(response.sendVendorSpecificHeaderOptions, 0,
@@ -1184,7 +1183,6 @@ OCStackResult BuildCollectionGroupActionCBORResponse(
 
             // Format the response.  Note this requires some info about the request
             response.requestHandle = ehRequest->requestHandle;
-            response.resourceHandle = ehRequest->resource;
             response.payload = (OCPayload*) payload;
             response.numSendVendorSpecificHeaderOptions = 0;
             memset(response.sendVendorSpecificHeaderOptions, 0,
@@ -1364,7 +1362,6 @@ OCStackResult BuildCollectionGroupActionCBORResponse(
 
             // Format the response.  Note this requires some info about the request
             response.requestHandle = ehRequest->requestHandle;
-            response.resourceHandle = ehRequest->resource;
             response.payload = (OCPayload*) payload;
             response.numSendVendorSpecificHeaderOptions = 0;
             memset(response.sendVendorSpecificHeaderOptions, 0,
index b07b88d..2a34eff 100644 (file)
@@ -421,8 +421,7 @@ OCStackResult SendKeepAliveResponse(OCServerRequest *request,
 
     OCEntityHandlerResponse ehResponse = { .ehResult = result, 
                                            .payload = (OCPayload*) payload, 
-                                           .requestHandle = request,
-                                           .resourceHandle = g_keepAliveHandle };
+                                           .requestHandle = request };
     OICStrcpy(ehResponse.resourceUri, sizeof(ehResponse.resourceUri), KEEPALIVE_RESOURCE_URI);
 
     // Send response message.
index 33faf9e..44e65ae 100644 (file)
@@ -3069,7 +3069,6 @@ static OCEntityHandlerResult DiagnosticPayloadRequest(OCEntityHandlerFlag flag,
     OCEntityHandlerResponse response;
     memset(&response, 0, sizeof(response));
     response.requestHandle = request->requestHandle;
-    response.resourceHandle = request->resource;
     response.ehResult = OC_EH_BAD_REQ;
     response.payload = (OCPayload*) OCDiagnosticPayloadCreate("message");
     EXPECT_TRUE(response.payload != NULL);
index 30ada5c..e1e127e 100644 (file)
@@ -620,7 +620,6 @@ namespace OC
             HeaderOptions serverHeaderOptions = pResponse->getHeaderOptions();
 
             response.requestHandle = pResponse->getRequestHandle();
-            response.resourceHandle = pResponse->getResourceHandle();
             response.ehResult = pResponse->getResponseResult();
 
             response.payload = reinterpret_cast<OCPayload*>(pResponse->getPayload());
index 28a4b26..240d759 100644 (file)
@@ -255,7 +255,6 @@ OCEntityHandlerResult CHPEntityHandler(OCEntityHandlerFlag flag,
                     ehResult = OC_EH_OK;
                     OCEntityHandlerResponse response =
                                 { .requestHandle = entityHandlerRequest->requestHandle,
-                                  .resourceHandle = entityHandlerRequest->resource,
                                   .ehResult = ehResult};
 
                     response.payload = (OCPayload *)CHPGetDiscoveryPayload();
@@ -295,8 +294,7 @@ void CHPHandleHttpResponse(const HttpResponse_t *httpResponse, void *context)
     }
 
     CHPRequest_t *ctxt = (CHPRequest_t *)context;
-    OCEntityHandlerResponse response = { .requestHandle = ctxt->requestHandle,
-                                         .resourceHandle = g_proxyHandle};
+    OCEntityHandlerResponse response = { .requestHandle = ctxt->requestHandle };
     response.persistentBufferFlag = 0;
 
     OCStackResult result = CHPGetOCCode(httpResponse->status, ctxt->method,
@@ -414,8 +412,7 @@ OCStackResult CHPHandleOCFRequest(const OCEntityHandlerRequest* requestInfo,
     HttpRequest_t httpRequest = { .httpMajor = 1,
                                   .httpMinor = 1};
 
-    OCEntityHandlerResponse response = { .requestHandle = requestInfo->requestHandle,
-                                         .resourceHandle = requestInfo->resource};
+    OCEntityHandlerResponse response = { .requestHandle = requestInfo->requestHandle };
     OCStackResult result = CHPGetHttpMethod(requestInfo->method, &httpRequest.method);
     if (OC_STACK_OK != result)
     {
index 2f67684..06010bd 100644 (file)
@@ -1545,7 +1545,6 @@ OCEntityHandlerResult OCEntityHandlerCb(OCEntityHandlerFlag flag,
 
         // Format the response.  Note this requires some info about the request
         response.requestHandle = entityHandlerRequest->requestHandle;
-        response.resourceHandle = entityHandlerRequest->resource;
         response.ehResult = ehRet;
         //response uses OCPaylod while all get,put methodes use OCRepPayload
         response.payload = (OCPayload*) (payload);
index 47a5d35..289febc 100644 (file)
@@ -702,7 +702,6 @@ OCStackResult NSProviderSendResponse(OCEntityHandlerRequest * entityHandlerReque
     memset(response.resourceUri, 0, sizeof response.resourceUri);\r
 \r
     response.requestHandle = entityHandlerRequest->requestHandle;\r
-    response.resourceHandle = entityHandlerRequest->resource;\r
     response.persistentBufferFlag = 0;\r
     response.ehResult = ehResult;\r
     response.payload = (OCPayload *) payload;\r
index cc558d3..5104bfa 100644 (file)
@@ -101,7 +101,6 @@ NSResult NSSendAccessPolicyResponse(OCEntityHandlerRequest *entityHandlerRequest
     OCRepPayloadSetPropString(payload, NS_ATTRIBUTE_TOPIC, NS_COLLECTION_TOPIC_URI);\r
 \r
     response.requestHandle = entityHandlerRequest->requestHandle;\r
-    response.resourceHandle = entityHandlerRequest->resource;\r
     response.persistentBufferFlag = 0;\r
     response.ehResult = OC_EH_OK;\r
     response.payload = (OCPayload *) payload;\r
index d3e44cc..2b9a55b 100644 (file)
@@ -335,7 +335,6 @@ NSResult NSSendTopicList(OCEntityHandlerRequest * entityHandlerRequest)
 
     NSOICFree(copyReq);
     response.requestHandle = entityHandlerRequest->requestHandle;
-    response.resourceHandle = entityHandlerRequest->resource;
     response.persistentBufferFlag = 0;
     response.ehResult = OC_EH_OK;
     response.payload = (OCPayload *) payload;