diff options
author | Reshetova, Elena <elena.reshetova@intel.com> | 2017-07-04 08:52:56 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-07-04 17:35:15 -0400 |
commit | 58951dde05fdac96a9d30805d8fa5d124dd4f92c (patch) | |
tree | b5e998f9a8415ffd543a98b9e1cc91c5ad1b06db /net/llc | |
parent | bf72acefebb459af3c805a386cd7e5456e3ad6ee (diff) |
net, llc: convert llc_sap.refcnt from atomic_t to refcount_t
refcount_t type and corresponding API should be
used instead of atomic_t when the variable is used as
a reference counter. This allows to avoid accidental
refcounter overflows that might lead to use-after-free
situations.
Signed-off-by: Elena Reshetova <elena.reshetova@intel.com>
Signed-off-by: Hans Liljestrand <ishkamiel@gmail.com>
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: David Windsor <dwindsor@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/llc')
-rw-r--r-- | net/llc/llc_core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/llc/llc_core.c b/net/llc/llc_core.c index 842851cef698..89041260784c 100644 --- a/net/llc/llc_core.c +++ b/net/llc/llc_core.c | |||
@@ -41,7 +41,7 @@ static struct llc_sap *llc_sap_alloc(void) | |||
41 | spin_lock_init(&sap->sk_lock); | 41 | spin_lock_init(&sap->sk_lock); |
42 | for (i = 0; i < LLC_SK_LADDR_HASH_ENTRIES; i++) | 42 | for (i = 0; i < LLC_SK_LADDR_HASH_ENTRIES; i++) |
43 | INIT_HLIST_NULLS_HEAD(&sap->sk_laddr_hash[i], i); | 43 | INIT_HLIST_NULLS_HEAD(&sap->sk_laddr_hash[i], i); |
44 | atomic_set(&sap->refcnt, 1); | 44 | refcount_set(&sap->refcnt, 1); |
45 | } | 45 | } |
46 | return sap; | 46 | return sap; |
47 | } | 47 | } |