diff options
author | Alexander Aring <alex.aring@gmail.com> | 2014-03-07 05:06:54 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-03-07 17:11:19 -0500 |
commit | 37147652cfaa20a87ead9bb04aec1834b40c5c97 (patch) | |
tree | a2fa74a0a64df5e52dae9c774771b5128a0a9b01 /net/ieee802154 | |
parent | d03e9d07adc8c3800b82c3f29fd0708f49a836dd (diff) |
6lowpan: reassembly: fix return of init function
This patch adds a missing return after fragmentation init. Otherwise we
register a sysctl interface and deregister it afterwards which makes no
sense.
Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ieee802154')
-rw-r--r-- | net/ieee802154/reassembly.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/net/ieee802154/reassembly.c b/net/ieee802154/reassembly.c index 1cc2336eb52c..bf06492e7d19 100644 --- a/net/ieee802154/reassembly.c +++ b/net/ieee802154/reassembly.c | |||
@@ -535,7 +535,7 @@ int __init lowpan_net_frag_init(void) | |||
535 | 535 | ||
536 | ret = lowpan_frags_sysctl_register(); | 536 | ret = lowpan_frags_sysctl_register(); |
537 | if (ret) | 537 | if (ret) |
538 | goto out; | 538 | return ret; |
539 | 539 | ||
540 | ret = register_pernet_subsys(&lowpan_frags_ops); | 540 | ret = register_pernet_subsys(&lowpan_frags_ops); |
541 | if (ret) | 541 | if (ret) |
@@ -550,9 +550,10 @@ int __init lowpan_net_frag_init(void) | |||
550 | lowpan_frags.frag_expire = lowpan_frag_expire; | 550 | lowpan_frags.frag_expire = lowpan_frag_expire; |
551 | lowpan_frags.secret_interval = 10 * 60 * HZ; | 551 | lowpan_frags.secret_interval = 10 * 60 * HZ; |
552 | inet_frags_init(&lowpan_frags); | 552 | inet_frags_init(&lowpan_frags); |
553 | |||
554 | return ret; | ||
553 | err_pernet: | 555 | err_pernet: |
554 | lowpan_frags_sysctl_unregister(); | 556 | lowpan_frags_sysctl_unregister(); |
555 | out: | ||
556 | return ret; | 557 | return ret; |
557 | } | 558 | } |
558 | 559 | ||