diff options
author | Jozsef Kadlecsik <kadlec@blackhole.kfki.hu> | 2010-01-24 15:49:16 +0100 |
---|---|---|
committer | Jozsef Kadlecsik <kadlec@blackhole.kfki.hu> | 2010-01-24 15:49:16 +0100 |
commit | 584e8d0604a55fcda68848bef82a2954d8af71f5 (patch) | |
tree | aef63f4901d1a68e65d84a23646ac487a809e24f /ipset_ipporthash.c | |
parent | c896b069e6c9cd229119a791c51d19e8f11c853c (diff) |
New version 4.2 released:v4.2
kernel:
- nethash and ipportnethash types counted every entry twice
which could produce bogus entries when listing/saving these types
of sets (bug reported by Husnu Demir)
userspace:
- Checking null entries when listing/saving hash types of sets
deleted because it's unnecessary and can mask possible errors.
Diffstat (limited to 'ipset_ipporthash.c')
-rw-r--r-- | ipset_ipporthash.c | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/ipset_ipporthash.c b/ipset_ipporthash.c index ce783ee..9249b2a 100644 --- a/ipset_ipporthash.c +++ b/ipset_ipporthash.c @@ -259,13 +259,11 @@ ipporthash_printips(struct set *set, void *data, u_int32_t len, while (offset < len) { ipptr = data + offset; - if (*ipptr) { - ip = (*ipptr>>16) + mysetdata->first_ip; - port = (uint16_t) *ipptr; - printf("%s,%s\n", - ip_tostring(ip, options), - port_tostring(port, options)); - } + ip = (*ipptr>>16) + mysetdata->first_ip; + port = (uint16_t) *ipptr; + printf("%s,%s\n", + ip_tostring(ip, options), + port_tostring(port, options)); offset += IPSET_VALIGN(sizeof(ip_set_ip_t), dont_align); } } @@ -296,13 +294,11 @@ ipporthash_saveips(struct set *set, void *data, u_int32_t len, while (offset < len) { ipptr = data + offset; - if (*ipptr) { - ip = (*ipptr>>16) + mysetdata->first_ip; - port = (uint16_t) *ipptr; - printf("-A %s %s,%s\n", set->name, - ip_tostring(ip, options), - port_tostring(port, options)); - } + ip = (*ipptr>>16) + mysetdata->first_ip; + port = (uint16_t) *ipptr; + printf("-A %s %s,%s\n", set->name, + ip_tostring(ip, options), + port_tostring(port, options)); offset += IPSET_VALIGN(sizeof(ip_set_ip_t), dont_align); } } |