From 12452d6a15fdc50ce53a1fe428fd2520979d7aa6 Mon Sep 17 00:00:00 2001 From: Janeks Jaunups Date: Tue, 6 May 2014 07:30:15 +0200 Subject: ipset: Handle missing leading zeros in ethernet address parser ipset would not parse ether addresses which are not exactly 17 characters long, for ex. 1:2:3:4:5:6, which is fixed in the patch. Signed-off-by: Jozsef Kadlecsik --- lib/parse.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) (limited to 'lib/parse.c') diff --git a/lib/parse.c b/lib/parse.c index 4db872e..be6e833 100644 --- a/lib/parse.c +++ b/lib/parse.c @@ -180,24 +180,27 @@ int ipset_parse_ether(struct ipset_session *session, enum ipset_opt opt, const char *str) { - unsigned int i = 0; + size_t len, p = 0, i = 0; unsigned char ether[ETH_ALEN]; assert(session); assert(opt == IPSET_OPT_ETHER); assert(str); - if (strlen(str) != ETH_ALEN * 3 - 1) + len = strlen(str); + + if (len > ETH_ALEN * 3 - 1) goto error; for (i = 0; i < ETH_ALEN; i++) { long number; char *end; - number = strtol(str + i * 3, &end, 16); + number = strtol(str + p, &end, 16); + p = end - str + 1; - if (end == str + i * 3 + 2 && - (*end == ':' || *end == '\0') && + if (((*end == ':' && i < ETH_ALEN - 1) || + (*end == '\0' && i == ETH_ALEN - 1)) && number >= 0 && number <= 255) ether[i] = number; else -- cgit v1.2.3