Fixed memory leak on ocstack.c
authorJaewook Jung <jw0213.jung@samsung.com>
Mon, 27 Mar 2017 08:26:49 +0000 (17:26 +0900)
committerAshok Babu Channa <ashok.channa@samsung.com>
Tue, 28 Mar 2017 05:21:57 +0000 (05:21 +0000)
Fixed memory leak on ocstack.c detected by valgrind

Change-Id: Idabf6cc8d0752663f2116f1c543d5879e29bf882
Signed-off-by: Jaewook Jung <jw0213.jung@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/18187
Tested-by: jenkins-iotivity <jenkins@iotivity.org>
Reviewed-by: Dan Mihai <Daniel.Mihai@microsoft.com>
Reviewed-by: Ashok Babu Channa <ashok.channa@samsung.com>
resource/csdk/stack/src/ocstack.c

index ce57949..988d7b2 100755 (executable)
@@ -1652,6 +1652,7 @@ void OCHandleResponse(const CAEndpoint_t* endPoint, const CAResponseInfo_t* resp
                     {
                         OIC_LOG_V(ERROR, TAG, "Unknown Payload type in Discovery: %d %s",
                                 cbNode->method, cbNode->requestUri);
+                        OICFree(response);
                         return;
                     }
                 }
@@ -1704,6 +1705,7 @@ void OCHandleResponse(const CAEndpoint_t* endPoint, const CAResponseInfo_t* resp
                     {
                         OIC_LOG(ERROR, TAG, "Error converting payload");
                         OCPayloadDestroy(response->payload);
+                        OICFree(response);
                         return;
                     }
 
@@ -1718,6 +1720,7 @@ void OCHandleResponse(const CAEndpoint_t* endPoint, const CAResponseInfo_t* resp
                         {
                             OIC_LOG(ERROR, TAG, "failed at map zone-id for link-local address");
                             OCPayloadDestroy(response->payload);
+                            OICFree(response);
                             return;
                         }
                     }