diff options
| author | Luciano Coelho <luciano.coelho@nokia.com> | 2010-07-15 17:12:56 +0200 |
|---|---|---|
| committer | Patrick McHardy <kaber@trash.net> | 2010-07-15 17:12:56 +0200 |
| commit | 67195a8c8a03d12994e91315e49e3d78c51a385a (patch) | |
| tree | dc1da66cd8fdf2af78a474bab3f3267406ba4220 | |
| parent | 127647892c7cac85baf8da62ed21232baa60f1c9 (diff) | |
extensions: libxt_IDLETIMER: use xtables_param_act when checking options
This patch changes custom error messages for illegal options into the
default iptables messages, by using xtables_param_act().
Signed-off-by: Luciano Coelho <luciano.coelho@nokia.com>
Signed-off-by: Patrick McHardy <kaber@trash.net>
| -rw-r--r-- | extensions/libxt_IDLETIMER.c | 15 | ||||
| -rw-r--r-- | extensions/libxt_IDLETIMER.man | 3 |
2 files changed, 8 insertions, 10 deletions
diff --git a/extensions/libxt_IDLETIMER.c b/extensions/libxt_IDLETIMER.c index 565f8e39..c931d0ee 100644 --- a/extensions/libxt_IDLETIMER.c +++ b/extensions/libxt_IDLETIMER.c @@ -60,23 +60,20 @@ static int idletimer_tg_parse(int c, char **argv, int invert, switch (c) { case 't': - if (*flags & IDLETIMER_TG_OPT_TIMEOUT) - xtables_error(PARAMETER_PROBLEM, - "Cannot specify timeout more than once"); + xtables_param_act(XTF_ONLY_ONCE, "IDLETIMER", "--timeout", + *flags & IDLETIMER_TG_OPT_TIMEOUT); info->timeout = atoi(optarg); *flags |= IDLETIMER_TG_OPT_TIMEOUT; break; case 'l': - if (*flags & IDLETIMER_TG_OPT_LABEL) - xtables_error(PARAMETER_PROBLEM, - "Cannot specify label more than once"); + xtables_param_act(XTF_ONLY_ONCE, "IDLETIMER", "--label", + *flags & IDLETIMER_TG_OPT_TIMEOUT); if (strlen(optarg) > MAX_IDLETIMER_LABEL_SIZE - 1) - xtables_error(PARAMETER_PROBLEM, - "Maximum label length is %u for --label", - MAX_IDLETIMER_LABEL_SIZE - 1); + xtables_param_act(XTF_BAD_VALUE, "IDLETIMER", "--label", + optarg); strcpy(info->label, optarg); *flags |= IDLETIMER_TG_OPT_LABEL; diff --git a/extensions/libxt_IDLETIMER.man b/extensions/libxt_IDLETIMER.man index 3266a448..e3c91cea 100644 --- a/extensions/libxt_IDLETIMER.man +++ b/extensions/libxt_IDLETIMER.man @@ -16,4 +16,5 @@ userspace, which can then decide what to do (eg. disconnect to save power). This is the time in seconds that will trigger the notification. .TP \fB\-\-label\fP \fIstring\fP -This is a unique identifier for the timer. +This is a unique identifier for the timer. The maximum length for the +label string is 27 characters. |