net, xfrm: convert sec_path.refcnt from atomic_t to refcount_t
authorReshetova, Elena <elena.reshetova@intel.com>
Tue, 4 Jul 2017 12:53:23 +0000 (15:53 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 4 Jul 2017 21:35:18 +0000 (22:35 +0100)
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>
include/net/xfrm.h
net/xfrm/xfrm_input.c

index e1bd1de2d66ac9f06961e6bf1b5bb3736ed4c13e..c0916ab18d3288033500e7a4953ef481176adaa0 100644 (file)
@@ -1030,7 +1030,7 @@ struct xfrm_offload {
 };
 
 struct sec_path {
-       atomic_t                refcnt;
+       refcount_t              refcnt;
        int                     len;
        int                     olen;
 
@@ -1051,7 +1051,7 @@ static inline struct sec_path *
 secpath_get(struct sec_path *sp)
 {
        if (sp)
-               atomic_inc(&sp->refcnt);
+               refcount_inc(&sp->refcnt);
        return sp;
 }
 
@@ -1060,7 +1060,7 @@ void __secpath_destroy(struct sec_path *sp);
 static inline void
 secpath_put(struct sec_path *sp)
 {
-       if (sp && atomic_dec_and_test(&sp->refcnt))
+       if (sp && refcount_dec_and_test(&sp->refcnt))
                __secpath_destroy(sp);
 }
 
index 9de4b1dbc0aea3f768a8d98eeab456c0c8e0fa61..923205e279f72dbab95cee827987feccc690c678 100644 (file)
@@ -116,7 +116,7 @@ struct sec_path *secpath_dup(struct sec_path *src)
                for (i = 0; i < sp->len; i++)
                        xfrm_state_hold(sp->xvec[i]);
        }
-       atomic_set(&sp->refcnt, 1);
+       refcount_set(&sp->refcnt, 1);
        return sp;
 }
 EXPORT_SYMBOL(secpath_dup);
@@ -126,7 +126,7 @@ int secpath_set(struct sk_buff *skb)
        struct sec_path *sp;
 
        /* Allocate new secpath or COW existing one. */
-       if (!skb->sp || atomic_read(&skb->sp->refcnt) != 1) {
+       if (!skb->sp || refcount_read(&skb->sp->refcnt) != 1) {
                sp = secpath_dup(skb->sp);
                if (!sp)
                        return -ENOMEM;