From f559820c7a2caaa3539d23fd495502bdc9baa155 Mon Sep 17 00:00:00 2001 From: WANG Cong Date: Mon, 7 Jan 2008 22:34:29 -0800 Subject: [PATCH] --- yaml --- r: 78685 b: refs/heads/master c: 64c31b3f76482bb64459e786f9eca3bd0164d153 h: refs/heads/master i: 78683: 29c54e2388008d902b3100e449ba5d01e1b0cf42 v: v3 --- [refs] | 2 +- trunk/include/net/xfrm.h | 4 ++-- trunk/net/key/af_key.c | 6 ++---- trunk/net/xfrm/xfrm_policy.c | 4 ++-- trunk/net/xfrm/xfrm_user.c | 2 +- 5 files changed, 8 insertions(+), 10 deletions(-) diff --git a/[refs] b/[refs] index 7efa12725bb2..43bc98194104 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: d66e37a99d323012165ce91fd5c4518e2fcea0c5 +refs/heads/master: 64c31b3f76482bb64459e786f9eca3bd0164d153 diff --git a/trunk/include/net/xfrm.h b/trunk/include/net/xfrm.h index a419a4372e21..5ebb9ba479b1 100644 --- a/trunk/include/net/xfrm.h +++ b/trunk/include/net/xfrm.h @@ -626,12 +626,12 @@ static inline void xfrm_pol_hold(struct xfrm_policy *policy) atomic_inc(&policy->refcnt); } -extern void __xfrm_policy_destroy(struct xfrm_policy *policy); +extern void xfrm_policy_destroy(struct xfrm_policy *policy); static inline void xfrm_pol_put(struct xfrm_policy *policy) { if (atomic_dec_and_test(&policy->refcnt)) - __xfrm_policy_destroy(policy); + xfrm_policy_destroy(policy); } #ifdef CONFIG_XFRM_SUB_POLICY diff --git a/trunk/net/key/af_key.c b/trunk/net/key/af_key.c index 76dcd882f87b..16b72b5570c3 100644 --- a/trunk/net/key/af_key.c +++ b/trunk/net/key/af_key.c @@ -2291,8 +2291,7 @@ static int pfkey_spdadd(struct sock *sk, struct sk_buff *skb, struct sadb_msg *h return 0; out: - security_xfrm_policy_free(xp); - kfree(xp); + xfrm_policy_destroy(xp); return err; } @@ -3236,8 +3235,7 @@ static struct xfrm_policy *pfkey_compile_policy(struct sock *sk, int opt, return xp; out: - security_xfrm_policy_free(xp); - kfree(xp); + xfrm_policy_destroy(xp); return NULL; } diff --git a/trunk/net/xfrm/xfrm_policy.c b/trunk/net/xfrm/xfrm_policy.c index d83227baaa09..534b29eb46fe 100644 --- a/trunk/net/xfrm/xfrm_policy.c +++ b/trunk/net/xfrm/xfrm_policy.c @@ -221,7 +221,7 @@ EXPORT_SYMBOL(xfrm_policy_alloc); /* Destroy xfrm_policy: descendant resources must be released to this moment. */ -void __xfrm_policy_destroy(struct xfrm_policy *policy) +void xfrm_policy_destroy(struct xfrm_policy *policy) { BUG_ON(!policy->dead); @@ -233,7 +233,7 @@ void __xfrm_policy_destroy(struct xfrm_policy *policy) security_xfrm_policy_free(policy); kfree(policy); } -EXPORT_SYMBOL(__xfrm_policy_destroy); +EXPORT_SYMBOL(xfrm_policy_destroy); static void xfrm_policy_gc_kill(struct xfrm_policy *policy) { diff --git a/trunk/net/xfrm/xfrm_user.c b/trunk/net/xfrm/xfrm_user.c index 6424e5360511..35fc16ae50ac 100644 --- a/trunk/net/xfrm/xfrm_user.c +++ b/trunk/net/xfrm/xfrm_user.c @@ -1043,7 +1043,7 @@ static struct xfrm_policy *xfrm_policy_construct(struct xfrm_userpolicy_info *p, return xp; error: *errp = err; - kfree(xp); + xfrm_policy_destroy(xp); return NULL; }