Check validity before free-ing options pointer.

Free pointer only if pointer is valid decided on basis of
number of options preset in the header.

Change-Id: I21666e4f694652120ed164ffc763b0aeeb0ea951
Signed-off-by: Mandeep Shetty <mandeep.shetty@intel.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/476
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Sashi Penta <sashi.kumar.penta@intel.com>
Reviewed-by: Sudarshan Prasad <sudarshan.prasad@intel.com>

diff --git a/resource/csdk/connectivity/src/caremotehandler.c b/resource/csdk/connectivity/src/caremotehandler.c
index 2e67030..76c3428 100644 (file)
--- a/resource/csdk/connectivity/src/caremotehandler.c
+++ b/resource/csdk/connectivity/src/caremotehandler.c
@@ -451,8 +451,10 @@ void CADestroyRequestInfoInternal(CARequestInfo_t *rep)
     OICFree((char *) rep->info.token);
 
     // free options field
-    OICFree((CAHeaderOption_t *) rep->info.options);
-
+    if (rep->info.options != NULL && rep->info.numOptions)
+    {
+        OICFree((CAHeaderOption_t *) rep->info.options);
+    }
     // free payload field
     OICFree((char *) rep->info.payload);