diff options
| author | Samuel Ortiz <sameo@linux.intel.com> | 2012-06-28 16:41:57 +0200 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2012-07-09 16:42:21 -0400 |
| commit | a831b9132065e1aa18acd7910d5f9c164a2f350f (patch) | |
| tree | 9cbaf7e194641e24e8e2f95e5d6f14cce9b7ff88 /net | |
| parent | 1550bf2d59a5ab74c5c14f2e65ed83516ca7d74d (diff) | |
| download | linux-a831b9132065e1aa18acd7910d5f9c164a2f350f.tar.gz linux-a831b9132065e1aa18acd7910d5f9c164a2f350f.tar.bz2 linux-a831b9132065e1aa18acd7910d5f9c164a2f350f.zip | |
NFC: Do not return EBUSY when stopping a poll that's already stopped
We check for the polling flag before checking if the netlink PID caller
match.
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/nfc/netlink.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/net/nfc/netlink.c b/net/nfc/netlink.c index 03c31db38f12..99bc6f7faa7b 100644 --- a/net/nfc/netlink.c +++ b/net/nfc/netlink.c @@ -634,6 +634,15 @@ static int nfc_genl_stop_poll(struct sk_buff *skb, struct genl_info *info) if (!dev) return -ENODEV; + device_lock(&dev->dev); + + if (!dev->polling) { + device_unlock(&dev->dev); + return -EINVAL; + } + + device_unlock(&dev->dev); + mutex_lock(&dev->genl_data.genl_data_mutex); if (dev->genl_data.poll_req_pid != info->snd_pid) { |