diff options
| author | Arnaud Ferir <arnaud.ferir@trusted-logic.com> | 2011-01-05 09:04:52 +0100 |
|---|---|---|
| committer | Nick Pelly <npelly@google.com> | 2011-01-18 15:28:26 -0800 |
| commit | 99e7261d709ad32e0203dced9f4a386c5f719adf (patch) | |
| tree | 0aa0475d8c73d73dbb02835222cecd0ef6b3de17 /src/phFriNfc_LlcpTransport.c | |
| parent | d9d57394761c70786f5058f82e7528bfaf1807a0 (diff) | |
| download | libnfc-nxp-99e7261d709ad32e0203dced9f4a386c5f719adf.tar.gz | |
Fixed parameters checking in LLCP.
Avoid NULL pointer dereferencing in some situations.
Change-Id: I020f0a5342acdc49f7f3804e9341ed0fc0f307ee
Diffstat (limited to 'src/phFriNfc_LlcpTransport.c')
| -rw-r--r-- | src/phFriNfc_LlcpTransport.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/phFriNfc_LlcpTransport.c b/src/phFriNfc_LlcpTransport.c index 22f1e32..e80357e 100644 --- a/src/phFriNfc_LlcpTransport.c +++ b/src/phFriNfc_LlcpTransport.c @@ -826,8 +826,12 @@ NFCSTATUS phFriNfc_LlcpTransport_Connect( phFriNfc_LlcpTransport_Socket_t* NFCSTATUS status = NFCSTATUS_SUCCESS; uint8_t i; - /* Check for NULL pointers */ - if(pLlcpSocket == NULL || pConnect_RspCb == NULL || pContext == NULL) + /* Check for NULL pointers */ + if(pConnect_RspCb == NULL) + { + status = PHNFCSTVAL(CID_FRI_NFC_LLCP_TRANSPORT, NFCSTATUS_INVALID_PARAMETER); + } + else if(pLlcpSocket == NULL || pConnect_RspCb == NULL || pContext == NULL) { status = PHNFCSTVAL(CID_FRI_NFC_LLCP_TRANSPORT, NFCSTATUS_INVALID_PARAMETER); /* Call the CB */ |