lwtunnel: fix memory leak
authorJiri Benc <jbenc@redhat.com>
Tue, 18 Aug 2015 16:41:13 +0000 (18:41 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Aug 2015 02:11:19 +0000 (19:11 -0700)
The built lwtunnel_state struct has to be freed after comparison.

Fixes: 571e722676fe3 ("ipv4: support for fib route lwtunnel encap attributes")
Signed-off-by: Jiri Benc <jbenc@redhat.com>
Acked-by: Roopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/lwtunnel.h
net/ipv4/fib_semantics.c

index e25b60e..34fd8f7 100644 (file)
@@ -36,6 +36,11 @@ struct lwtunnel_encap_ops {
 };
 
 #ifdef CONFIG_LWTUNNEL
+static inline void lwtstate_free(struct lwtunnel_state *lws)
+{
+       kfree(lws);
+}
+
 static inline struct lwtunnel_state *
 lwtstate_get(struct lwtunnel_state *lws)
 {
@@ -51,7 +56,7 @@ static inline void lwtstate_put(struct lwtunnel_state *lws)
                return;
 
        if (atomic_dec_and_test(&lws->refcnt))
-               kfree(lws);
+               lwtstate_free(lws);
 }
 
 static inline bool lwtunnel_output_redirect(struct lwtunnel_state *lwtstate)
Simple merge