diff options
| author | David S. Miller <davem@davemloft.net> | 2012-05-10 22:16:32 -0400 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2012-06-10 14:42:02 +0100 |
| commit | 9f6cf3abdb1e954db86f963569590444642495ff (patch) | |
| tree | ccf07daac4181ba0874b03a810ab13c72298813c /net/ipv4 | |
| parent | 7a735af2d50190b6c612509c6dcad1b5aa0be828 (diff) | |
| download | lwn-9f6cf3abdb1e954db86f963569590444642495ff.tar.gz lwn-9f6cf3abdb1e954db86f963569590444642495ff.zip | |
ipv4: Do not use dead fib_info entries.
[ Upstream commit dccd9ecc374462e5d6a5b8f8110415a86c2213d8 ]
Due to RCU lookups and RCU based release, fib_info objects can
be found during lookup which have fi->fib_dead set.
We must ignore these entries, otherwise we risk dereferencing
the parts of the entry which are being torn down.
Reported-by: Yevgen Pronenko <yevgen.pronenko@sonymobile.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'net/ipv4')
| -rw-r--r-- | net/ipv4/fib_trie.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/ipv4/fib_trie.c b/net/ipv4/fib_trie.c index 3ce23f9d9363..cd2d639541bc 100644 --- a/net/ipv4/fib_trie.c +++ b/net/ipv4/fib_trie.c @@ -1372,6 +1372,8 @@ static int check_leaf(struct fib_table *tb, struct trie *t, struct leaf *l, if (fa->fa_tos && fa->fa_tos != flp->flowi4_tos) continue; + if (fi->fib_dead) + continue; if (fa->fa_info->fib_scope < flp->flowi4_scope) continue; fib_alias_accessed(fa); |