vti6: Simplify error handling in module init and exit
authorMathias Krause <minipli@googlemail.com>
Fri, 9 May 2014 21:43:41 +0000 (23:43 +0200)
committerSteffen Klassert <steffen.klassert@secunet.com>
Thu, 26 Jun 2014 06:21:57 +0000 (08:21 +0200)
commite59d82fd33f7670cf67fd69cf684aa589ec8340a
treecb7747de194a9979831ca943dac1a00d331c10b7
parent644a918d20336a7deaa81f675c3c2f25bf3dafbb
vti6: Simplify error handling in module init and exit

The error handling in the module init and exit functions can be
shortened to safe us some code.

1/ Remove the code duplications in the init function, jump straight to
the existing cleanup code by adding some labels. Also give the error
message some more value by telling the reason why loading the module has
failed.

2/ Remove the error handling in the exit function as the only legitimate
reason xfrm6_protocol_deregister() might fail is inet6_del_protocol()
returning -1. That, in turn, means some other protocol handler had been
registered for this very protocol in the meantime. But that essentially
means we haven't been handling that protocol any more, anyway. What it
definitely means not is that we "can't deregister protocol". Therefore
just get rid of that bogus warning. It's plain wrong.

Signed-off-by: Mathias Krause <minipli@googlemail.com>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
net/ipv6/ip6_vti.c
This page took 0.028059 seconds and 5 git commands to generate.