diff options
author | Jan Engelhardt <jengelh@medozas.de> | 2010-03-21 04:05:56 +0100 |
---|---|---|
committer | Jan Engelhardt <jengelh@medozas.de> | 2010-03-25 16:56:09 +0100 |
commit | f95c74e33eff5e3fe9798e2dc0a7749150ea3f80 (patch) | |
tree | 4e716e6aa0409d2cdcbfbcb7314af93a07ef6585 /net/netfilter/xt_connmark.c | |
parent | 4a5a5c73b7cfee46a0b1411903cfa0dea532deec (diff) | |
download | op-kernel-dev-f95c74e33eff5e3fe9798e2dc0a7749150ea3f80.zip op-kernel-dev-f95c74e33eff5e3fe9798e2dc0a7749150ea3f80.tar.gz |
netfilter: xtables: shorten up return clause
The return value of nf_ct_l3proto_get can directly be returned even in
the case of success.
Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
Diffstat (limited to 'net/netfilter/xt_connmark.c')
-rw-r--r-- | net/netfilter/xt_connmark.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/net/netfilter/xt_connmark.c b/net/netfilter/xt_connmark.c index 71e38a1..ae10154 100644 --- a/net/netfilter/xt_connmark.c +++ b/net/netfilter/xt_connmark.c @@ -79,12 +79,10 @@ static int connmark_tg_check(const struct xt_tgchk_param *par) int ret; ret = nf_ct_l3proto_try_module_get(par->family); - if (ret < 0) { + if (ret < 0) pr_info("cannot load conntrack support for proto=%u\n", par->family); - return ret; - } - return 0; + return ret; } static void connmark_tg_destroy(const struct xt_tgdtor_param *par) @@ -111,12 +109,10 @@ static int connmark_mt_check(const struct xt_mtchk_param *par) int ret; ret = nf_ct_l3proto_try_module_get(par->family); - if (ret < 0) { + if (ret < 0) pr_info("cannot load conntrack support for proto=%u\n", par->family); - return ret; - } - return 0; + return ret; } static void connmark_mt_destroy(const struct xt_mtdtor_param *par) |