diff options
| author | Jan Engelhardt | 2010-03-19 17:32:59 +0100 |
|---|---|---|
| committer | Jan Engelhardt | 2010-03-25 16:56:09 +0100 |
| commit | 4a5a5c73b7cfee46a0b1411903cfa0dea532deec (patch) | |
| tree | 3f7ef0def4adf3a01e15ab3a2a7231967b87bfee /net/netfilter/xt_time.c | |
| parent | netfilter: xtables: change targets to return error code (diff) | |
| download | kernel-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_time.c')
| -rw-r--r-- | net/netfilter/xt_time.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netfilter/xt_time.c b/net/netfilter/xt_time.c index 8dde5e51ff19..d8556fdda440 100644 --- a/net/netfilter/xt_time.c +++ b/net/netfilter/xt_time.c @@ -225,7 +225,7 @@ static int time_mt_check(const struct xt_mtchk_param *par) info->daytime_stop > XT_TIME_MAX_DAYTIME) { pr_info("invalid argument - start or " "stop time greater than 23:59:59\n"); - return -EINVAL; + return -EDOM; } return 0; |