Skip to content

Commit

Permalink
NFC: Fix next target_idx type and rename for clarity
Browse files Browse the repository at this point in the history
Signed-off-by: Eric Lapuyade <eric.lapuyade@intel.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
  • Loading branch information
Eric Lapuyade authored and John W. Linville committed Apr 12, 2012
1 parent c4fbb65 commit 01ae0ee
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion include/net/nfc/nfc.h
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ struct nfc_genl_data {

struct nfc_dev {
unsigned idx;
unsigned target_idx;
u32 target_next_idx;
struct nfc_target *targets;
int n_targets;
int targets_generation;
Expand Down
2 changes: 1 addition & 1 deletion net/nfc/core.c
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@ int nfc_targets_found(struct nfc_dev *dev,
dev->polling = false;

for (i = 0; i < n_targets; i++)
targets[i].idx = dev->target_idx++;
targets[i].idx = dev->target_next_idx++;

spin_lock_bh(&dev->targets_lock);

Expand Down
4 changes: 2 additions & 2 deletions net/nfc/rawsock.c
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,8 @@ static int rawsock_connect(struct socket *sock, struct sockaddr *_addr,
goto error;
}

if (addr->target_idx > dev->target_idx - 1 ||
addr->target_idx < dev->target_idx - dev->n_targets) {
if (addr->target_idx > dev->target_next_idx - 1 ||
addr->target_idx < dev->target_next_idx - dev->n_targets) {
rc = -EINVAL;
goto error;
}
Expand Down

0 comments on commit 01ae0ee

Please sign in to comment.