diff options
author | WANG Cong <xiyou.wangcong@gmail.com> | 2015-03-25 17:45:02 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-03-29 15:12:37 -0400 |
commit | 85b9909272f128757dcb6d053ca12e15931747d4 (patch) | |
tree | 13f6be2a38fc787a3fa3137bc32c2c5ef023936c /net/ipv6 | |
parent | 7145074b374dd90a4512fe52c062b336b724f276 (diff) |
fib6: install fib6 ops in the last step
We should not commit the new ops until we finish
all the setup, otherwise we have to NULL it on failure.
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6')
-rw-r--r-- | net/ipv6/fib6_rules.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/net/ipv6/fib6_rules.c b/net/ipv6/fib6_rules.c index 27ca79682efb..273eb26cd6d4 100644 --- a/net/ipv6/fib6_rules.c +++ b/net/ipv6/fib6_rules.c | |||
@@ -299,19 +299,16 @@ static int __net_init fib6_rules_net_init(struct net *net) | |||
299 | ops = fib_rules_register(&fib6_rules_ops_template, net); | 299 | ops = fib_rules_register(&fib6_rules_ops_template, net); |
300 | if (IS_ERR(ops)) | 300 | if (IS_ERR(ops)) |
301 | return PTR_ERR(ops); | 301 | return PTR_ERR(ops); |
302 | net->ipv6.fib6_rules_ops = ops; | ||
303 | |||
304 | 302 | ||
305 | err = fib_default_rule_add(net->ipv6.fib6_rules_ops, 0, | 303 | err = fib_default_rule_add(ops, 0, RT6_TABLE_LOCAL, 0); |
306 | RT6_TABLE_LOCAL, 0); | ||
307 | if (err) | 304 | if (err) |
308 | goto out_fib6_rules_ops; | 305 | goto out_fib6_rules_ops; |
309 | 306 | ||
310 | err = fib_default_rule_add(net->ipv6.fib6_rules_ops, | 307 | err = fib_default_rule_add(ops, 0x7FFE, RT6_TABLE_MAIN, 0); |
311 | 0x7FFE, RT6_TABLE_MAIN, 0); | ||
312 | if (err) | 308 | if (err) |
313 | goto out_fib6_rules_ops; | 309 | goto out_fib6_rules_ops; |
314 | 310 | ||
311 | net->ipv6.fib6_rules_ops = ops; | ||
315 | out: | 312 | out: |
316 | return err; | 313 | return err; |
317 | 314 | ||