| commit | e5a1f3787f4b43e828de2818faa6a0f658b87a63 | |
| author | Steffen Klassert <steffen.klassert@secunet.com> | |
| Tue, 15 Mar 2011 21:12:49 +0000 (15 21:12 +0000) | ||
| committer | Greg Kroah-Hartman <gregkh@suse.de> | |
| Thu, 21 Apr 2011 21:32:34 +0000 (21 14:32 -0700) | ||
| tree | 37e3748d3101cdb40cd28ee52a46d51ee509f0e7 | treesnapshot (tar.gz zip) |
| parent | edc45d1e743b8b5a9366b7cafb53c80f4267fe1f | commitdiff |
xfrm: Refcount destination entry on xfrm_lookup
[ Upstream commit fbd5060875d25f7764fd1c3d35b83a8ed1d88d7b ]
We return a destination entry without refcount if a socket
policy is found in xfrm_lookup. This triggers a warning on
a negative refcount when freeeing this dst entry. So take
a refcount in this case to fix it.
This refcount was forgotten when xfrm changed to cache bundles
instead of policies for outgoing flows.
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Acked-by: Timo Teräs <timo.teras@iki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
[ Upstream commit fbd5060875d25f7764fd1c3d35b83a8ed1d88d7b ]
We return a destination entry without refcount if a socket
policy is found in xfrm_lookup. This triggers a warning on
a negative refcount when freeeing this dst entry. So take
a refcount in this case to fix it.
This refcount was forgotten when xfrm changed to cache bundles
instead of policies for outgoing flows.
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Acked-by: Timo Teräs <timo.teras@iki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| net/xfrm/xfrm_policy.c | diffblobblamehistory |