summaryrefslogtreecommitdiffstats
path: root/drivers/nfc
diff options
context:
space:
mode:
authorWaldemar Rymarkiewicz <waldemar.rymarkiewicz@tieto.com>2012-12-21 10:39:01 +0100
committerSamuel Ortiz <sameo@linux.intel.com>2013-01-10 00:51:55 +0100
commit8a0ecfe74b3692258311d084d31613516828905f (patch)
tree341d2693e9a58a9f932ef0f8bfc1a00845763f31 /drivers/nfc
parent390a1bd8538132186ddb679cafe9e75b7ef7e2d2 (diff)
downloadop-kernel-dev-8a0ecfe74b3692258311d084d31613516828905f.zip
op-kernel-dev-8a0ecfe74b3692258311d084d31613516828905f.tar.gz
NFC: pn533: Fix missing parenthesis
This is a quite critical patch as it fixes potential reference to undefined general_bytes which were never set correctly on target activation due to missing parenthesis. Signed-off-by: Waldemar Rymarkiewicz <waldemar.rymarkiewicz@tieto.com> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Diffstat (limited to 'drivers/nfc')
-rw-r--r--drivers/nfc/pn533.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/nfc/pn533.c b/drivers/nfc/pn533.c
index 31a5b3b..f1702f9 100644
--- a/drivers/nfc/pn533.c
+++ b/drivers/nfc/pn533.c
@@ -1724,9 +1724,10 @@ static int pn533_activate_target_nfcdep(struct pn533 *dev)
rsp = (struct pn533_cmd_activate_response *)resp->data;
rc = rsp->status & PN533_CMD_RET_MASK;
- if (rc != PN533_CMD_RET_SUCCESS)
+ if (rc != PN533_CMD_RET_SUCCESS) {
dev_kfree_skb(resp);
return -EIO;
+ }
/* ATR_RES general bytes are located at offset 16 */
gt_len = resp->len - 16;
OpenPOWER on IntegriCloud