aboutsummaryrefslogtreecommitdiffstats
path: root/net/ipv4/ip_vti.c
diff options
context:
space:
mode:
authorMathias Krause <minipli@googlemail.com>2014-05-09 23:43:41 +0200
committerSteffen Klassert <steffen.klassert@secunet.com>2014-06-26 08:21:57 +0200
commite59d82fd33f7670cf67fd69cf684aa589ec8340a (patch)
treecb7747de194a9979831ca943dac1a00d331c10b7 /net/ipv4/ip_vti.c
parentenic: Make dummy rfs functions inline to fix !CONFIG_RFS_ACCEL build (diff)
downloadlinux-dev-e59d82fd33f7670cf67fd69cf684aa589ec8340a.tar.xz
linux-dev-e59d82fd33f7670cf67fd69cf684aa589ec8340a.zip
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>
Diffstat (limited to 'net/ipv4/ip_vti.c')
0 files changed, 0 insertions, 0 deletions