diff options
| author | Patrick McHardy <kaber@trash.net> | 2009-11-20 14:58:11 +0100 |
|---|---|---|
| committer | Patrick McHardy <kaber@trash.net> | 2009-11-20 14:58:11 +0100 |
| commit | f294f843473718f8d32745600b9a97c0b799e7c5 (patch) | |
| tree | 221f6a7f0e39c95a50fd7e08068b03a74397a15d | |
| parent | 588b615bc78ddef3752f356d1e243129c4dbba96 (diff) | |
conntrack: fix --expires parsing
Using ranges in --ctexpire results in a parsing error:
conntrack: Bad value for "--expires" option: "1:1000"
The first value is parsed twice, after which the end pointer doesn't
point to the expected '\0' but to the colon.
Signed-off-by: Patrick McHardy <kaber@trash.net>
| -rw-r--r-- | extensions/libxt_conntrack.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extensions/libxt_conntrack.c b/extensions/libxt_conntrack.c index d30871fc..5ca734d2 100644 --- a/extensions/libxt_conntrack.c +++ b/extensions/libxt_conntrack.c @@ -273,7 +273,7 @@ conntrack_ps_expires(struct xt_conntrack_mtinfo2 *info, const char *s) xtables_param_act(XTF_BAD_VALUE, "conntrack", "--expires", s); max = min; if (*end == ':') - if (!xtables_strtoui(s, &end, &max, 0, UINT32_MAX)) + if (!xtables_strtoui(end + 1, &end, &max, 0, UINT32_MAX)) xtables_param_act(XTF_BAD_VALUE, "conntrack", "--expires", s); if (*end != '\0') xtables_param_act(XTF_BAD_VALUE, "conntrack", "--expires", s); |