diff mbox series

[4.9,32/32] NFC: fix possible resource leak

Message ID	20210202132943.304393716@linuxfoundation.org
State	Superseded
Headers	show Return-Path: <stable-owner@kernel.org> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, stable@vger.kernel.org, Pan Bian <bianpan2016@163.com>, Jakub Kicinski <kuba@kernel.org> Subject: [PATCH 4.9 32/32] NFC: fix possible resource leak Date: Tue, 2 Feb 2021 14:38:55 +0100 Message-Id: <20210202132943.304393716@linuxfoundation.org> In-Reply-To: <20210202132942.035179752@linuxfoundation.org> References: <20210202132942.035179752@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	None \| expand [4.9,02/32] wext: fix NULL-ptr-dereference with cfg80211s lack of commit() [4.9,03/32] net: usb: qmi_wwan: added support for Thales Cinterion PLSx3 modem family [4.9,04/32] y2038: futex: Move compat implementation into futex.c [4.9,05/32] futex: Move futex exit handling into futex code [4.9,06/32] futex: Replace PF_EXITPIDONE with a state [4.9,07/32] exit/exec: Seperate mm_release() [4.9,08/32] futex: Split futex_mm_release() for exit/exec [4.9,09/32] futex: Set task::futex_state to DEAD right after handling futex exit [4.9,10/32] futex: Mark the begin of futex exit explicitly [4.9,11/32] futex: Sanitize exit state handling [4.9,12/32] futex: Provide state handling for exec() as well [4.9,13/32] futex: Add mutex around futex exit [4.9,14/32] futex: Provide distinct return value when owner is exiting [4.9,15/32] futex: Prevent exit livelock [4.9,16/32] KVM: x86/pmu: Fix HW_REF_CPU_CYCLES event pseudo-encoding in intel_arch_events[] [4.9,17/32] KVM: x86: get smi pending status correctly [4.9,18/32] leds: trigger: fix potential deadlock with libata [4.9,19/32] mt7601u: fix kernel crash unplugging the device [4.9,20/32] mt7601u: fix rx buffer refcounting [4.9,21/32] ARM: imx: build suspend-imx6.S with arm instruction set [4.9,22/32] netfilter: nft_dynset: add timeout extension to template [4.9,23/32] xfrm: Fix oops in xfrm_replay_advance_bmp [4.9,24/32] RDMA/cxgb4: Fix the reported max_recv_sge value [4.9,25/32] iwlwifi: pcie: use jiffies for memory read spin time limit [4.9,26/32] iwlwifi: pcie: reschedule in long-running memory reads [4.9,27/32] mac80211: pause TX while changing interface type [4.9,28/32] can: dev: prevent potential information leak in can_fill_info() [4.9,29/32] iommu/vt-d: Gracefully handle DMAR units with no supported address widths [4.9,30/32] iommu/vt-d: Dont dereference iommu_device if IOMMU_API is not built [4.9,31/32] NFC: fix resource leak when target index is invalid [4.9,32/32] NFC: fix possible resource leak

Commit Message

Greg Kroah-Hartman Feb. 2, 2021, 1:38 p.m. UTC

From: Pan Bian <bianpan2016@163.com>

commit d8f923c3ab96dbbb4e3c22d1afc1dc1d3b195cd8 upstream.

Put the device to avoid resource leak on path that the polling flag is
invalid.

Fixes: a831b9132065 ("NFC: Do not return EBUSY when stopping a poll that's already stopped")
Signed-off-by: Pan Bian <bianpan2016@163.com>
Link: https://lore.kernel.org/r/20210121153745.122184-1-bianpan2016@163.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

---
 net/nfc/netlink.c |    1 +
 1 file changed, 1 insertion(+)

diff mbox series

Patch

--- a/net/nfc/netlink.c
+++ b/net/nfc/netlink.c
@@ -887,6 +887,7 @@  static int nfc_genl_stop_poll(struct sk_
 
 	if (!dev->polling) {
 		device_unlock(&dev->dev);
+		nfc_put_device(dev);
 		return -EINVAL;
 	}