summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorEric Biggers <ebiggers@google.com>2019-10-06 14:24:27 -0700
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2020-01-27 14:51:17 +0100
commit255aba7762bf83096d8437dc7c583ef9abea10a1 (patch)
tree7ea27471b3cd2cf5f30c2ccfe5aaead3c5e6c7ad /drivers
parent796d0f369aff2845b3623b8bec20002b1b376ca8 (diff)
llc: fix sk_buff refcounting in llc_conn_state_process()
[ Upstream commit 36453c852816f19947ca482a595dffdd2efa4965 ] If llc_conn_state_process() sees that llc_conn_service() put the skb on a list, it will drop one fewer references to it. This is wrong because the current behavior is that llc_conn_service() never consumes a reference to the skb. The code also makes the number of skb references being dropped conditional on which of ind_prim and cfm_prim are nonzero, yet neither of these affects how many references are *acquired*. So there is extra code that tries to fix this up by sometimes taking another reference. Remove the unnecessary/broken refcounting logic and instead just add an skb_get() before the only two places where an extra reference is actually consumed. Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2") Signed-off-by: Eric Biggers <ebiggers@google.com> Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers')
0 files changed, 0 insertions, 0 deletions