Resolved JIRA [IOT-1009]
authorsaurabh.s9 <saurabh.s9@samsung.com>
Wed, 23 Mar 2016 04:44:48 +0000 (10:14 +0530)
committerRandeep Singh <randeep.s@samsung.com>
Wed, 23 Mar 2016 12:51:52 +0000 (12:51 +0000)
Change-Id: Ia21ccd9c3e820aaf9eba1bdf352dbe8ec0ea65be
Signed-off-by: saurabh.s9 <saurabh.s9@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/6195
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Md. Imtiaz Hossain <imtiaz.h@samsung.com>
Reviewed-by: dongik Lee <dongik.lee@samsung.com>
Reviewed-by: Randeep Singh <randeep.s@samsung.com>
resource/csdk/security/provisioning/src/provisioningdatabasemanager.c

index 83e7b99..146f77d 100644 (file)
@@ -626,13 +626,13 @@ static OCStackResult getUUIDforId(int id, OicUuid_t *uid, bool *result)
 OCStackResult PDMGetLinkedDevices(const OicUuid_t *UUID, OCUuidList_t **UUIDLIST, size_t *numOfDevices)
 {
     CHECK_PDM_INIT(TAG);
-    if (NULL != *UUIDLIST)
+    if (NULL == UUID || NULL == numOfDevices || !UUIDLIST)
     {
-        OIC_LOG(ERROR, TAG, "Not null list will cause memory leak");
         return OC_STACK_INVALID_PARAM;
     }
-    if (NULL == UUID)
+    if (NULL != *UUIDLIST)
     {
+        OIC_LOG(ERROR, TAG, "Not null list will cause memory leak");
         return OC_STACK_INVALID_PARAM;
     }
     bool result = false;