summaryrefslogtreecommitdiffstats
path: root/net/netfilter/xt_HL.c
diff options
context:
space:
mode:
authorJan Engelhardt2010-03-19 17:32:59 +0100
committerJan Engelhardt2010-03-25 16:56:09 +0100
commit4a5a5c73b7cfee46a0b1411903cfa0dea532deec (patch)
tree3f7ef0def4adf3a01e15ab3a2a7231967b87bfee /net/netfilter/xt_HL.c
parentnetfilter: xtables: change targets to return error code (diff)
downloadkernel-qcow2-linux-4a5a5c73b7cfee46a0b1411903cfa0dea532deec.tar.gz
kernel-qcow2-linux-4a5a5c73b7cfee46a0b1411903cfa0dea532deec.tar.xz
kernel-qcow2-linux-4a5a5c73b7cfee46a0b1411903cfa0dea532deec.zip
netfilter: xtables: slightly better error reporting
When extended status codes are available, such as ENOMEM on failed allocations, or subsequent functions (e.g. nf_ct_get_l3proto), passing them up to userspace seems like a good idea compared to just always EINVAL. Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
Diffstat (limited to 'net/netfilter/xt_HL.c')
-rw-r--r--net/netfilter/xt_HL.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netfilter/xt_HL.c b/net/netfilter/xt_HL.c
index 7a47383ec723..77b99f732711 100644
--- a/net/netfilter/xt_HL.c
+++ b/net/netfilter/xt_HL.c
@@ -107,7 +107,7 @@ static int ttl_tg_check(const struct xt_tgchk_param *par)
if (info->mode > IPT_TTL_MAXMODE) {
pr_info("TTL: invalid or unknown mode %u\n", info->mode);
- return false;
+ return -EINVAL;
}
if (info->mode != IPT_TTL_SET && info->ttl == 0)
return -EINVAL;