aboutsummaryrefslogtreecommitdiffstats
path: root/net/core/fib_rules.c
diff options
context:
space:
mode:
authorGao feng <gaofeng@cn.fujitsu.com>2011-09-11 15:36:05 +0000
committerGreg Kroah-Hartman <gregkh@suse.de>2011-10-03 11:40:51 -0700
commitcbab190c501c8034b82e0dd9da7fdb4b75e08daa (patch)
tree906c1a75f88c366d3e85ab20caf6b4bb2acc238c /net/core/fib_rules.c
parent5ee858c9ab53a4f6b12a095a4982793d4f417f56 (diff)
downloadkernel_samsung_smdk4412-cbab190c501c8034b82e0dd9da7fdb4b75e08daa.zip
kernel_samsung_smdk4412-cbab190c501c8034b82e0dd9da7fdb4b75e08daa.tar.gz
kernel_samsung_smdk4412-cbab190c501c8034b82e0dd9da7fdb4b75e08daa.tar.bz2
fib:fix BUG_ON in fib_nl_newrule when add new fib rule
[ Upstream commit 561dac2d410ffac0b57a23b85ae0a623c1a076ca ] add new fib rule can cause BUG_ON happen the reproduce shell is ip rule add pref 38 ip rule add pref 38 ip rule add to 192.168.3.0/24 goto 38 ip rule del pref 38 ip rule add to 192.168.3.0/24 goto 38 ip rule add pref 38 then the BUG_ON will happen del BUG_ON and use (ctarget == NULL) identify whether this rule is unresolved Signed-off-by: Gao feng <gaofeng@cn.fujitsu.com> Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'net/core/fib_rules.c')
-rw-r--r--net/core/fib_rules.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/net/core/fib_rules.c b/net/core/fib_rules.c
index 008dc70..f39ef5c 100644
--- a/net/core/fib_rules.c
+++ b/net/core/fib_rules.c
@@ -384,8 +384,8 @@ static int fib_nl_newrule(struct sk_buff *skb, struct nlmsghdr* nlh, void *arg)
*/
list_for_each_entry(r, &ops->rules_list, list) {
if (r->action == FR_ACT_GOTO &&
- r->target == rule->pref) {
- BUG_ON(rtnl_dereference(r->ctarget) != NULL);
+ r->target == rule->pref &&
+ rtnl_dereference(r->ctarget) == NULL) {
rcu_assign_pointer(r->ctarget, rule);
if (--ops->unresolved_rules == 0)
break;