Fixed memory leak on ocpayload.c
authorhyuna0213.jo <hyuna0213.jo@samsung.com>
Tue, 16 May 2017 07:17:50 +0000 (16:17 +0900)
committerAshok Babu Channa <ashok.channa@samsung.com>
Sat, 20 May 2017 10:06:05 +0000 (10:06 +0000)
Fixed memory leak on ocpayload.c detected by static analysis

Change-Id: I3a8de43b8feff88509af6959f6b78b79eb8feb4c
Signed-off-by: hyuna0213.jo <hyuna0213.jo@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/19937
Reviewed-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-by: Dan Mihai <Daniel.Mihai@microsoft.com>
Tested-by: jenkins-iotivity <jenkins@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>
Reviewed-by: Jaewook Jung <jw0213.jung@samsung.com>
Reviewed-by: Ashok Babu Channa <ashok.channa@samsung.com>
resource/csdk/stack/src/ocpayload.c

index 86d650b..a0582fd 100644 (file)
@@ -1840,6 +1840,7 @@ static OCResourcePayload* OCCopyResource(const OCResource* res, uint16_t secureP
                                                             info->flags,
                                                             &matchedTps))
                     {
+                        OCDiscoveryResourceDestroy(pl);
                         return NULL;
                     }
 
@@ -1852,6 +1853,7 @@ static OCResourcePayload* OCCopyResource(const OCResource* res, uint16_t secureP
                             OICCalloc(1, sizeof(OCEndpointPayload));
                         if (!tmpNode)
                         {
+                            OCDiscoveryResourceDestroy(pl);
                             return NULL;
                         }