Correcting error case return value
authorsahil bansal <sahil.bansal@samsung.com>
Thu, 17 Mar 2016 11:20:01 +0000 (16:50 +0530)
committerRandeep Singh <randeep.s@samsung.com>
Sat, 19 Mar 2016 13:51:13 +0000 (13:51 +0000)
Change-Id: If1b1e8e0674ab37461f95a8fae0a22b026fe4ce6
Signed-off-by: sahil bansal <sahil.bansal@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/6011
Reviewed-by: Joonghwan Lee <jh05.lee@samsung.com>
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Randeep Singh <randeep.s@samsung.com>
resource/csdk/stack/src/ocstack.c

index 1302699..6e580a7 100644 (file)
@@ -3902,12 +3902,17 @@ OCStackResult OCDoDirectPairing(OCDPDev_t* peer, OCPrm_t pmSel, char *pinNumber,
                                                      OCDirectPairingCB resultCallback)
 {
     OIC_LOG(INFO, TAG, "Start OCDoDirectPairing");
-    if(NULL ==  peer || NULL == resultCallback)
+    if(NULL ==  peer)
     {
         OIC_LOG(ERROR, TAG, "Invalid parameters");
         return OC_STACK_INVALID_PARAM;
     }
 
+    if(NULL == resultCallback)
+    {
+        OIC_LOG(ERROR, TAG, "Invalid parameters");
+        return OC_STACK_INVALID_CALLBACK;
+    }
     gDirectpairingCallback = resultCallback;
     return DPDirectPairing((OCDirectPairingDev_t*)peer, (OicSecPrm_t)pmSel,
                                            pinNumber, DirectPairingCB);