summaryrefslogtreecommitdiffstats
path: root/libiptc
diff options
context:
space:
mode:
author/C=DE/ST=Berlin/L=Berlin/O=Netfilter Project/OU=Development/CN=gandalf/emailAddress=gandalf@netfilter.org </C=DE/ST=Berlin/L=Berlin/O=Netfilter Project/OU=Development/CN=gandalf/emailAddress=gandalf@netfilter.org>2005-02-04 07:38:02 +0000
committer/C=DE/ST=Berlin/L=Berlin/O=Netfilter Project/OU=Development/CN=gandalf/emailAddress=gandalf@netfilter.org </C=DE/ST=Berlin/L=Berlin/O=Netfilter Project/OU=Development/CN=gandalf/emailAddress=gandalf@netfilter.org>2005-02-04 07:38:02 +0000
commit6cf0411600b405b71dcff976670df8f8d2fe8622 (patch)
treefd866fa0a50cc6a0141ed4aea3f2b7db3b686b32 /libiptc
parent5244bd949f12e7c1dba979fad52bcb5ad6dca09a (diff)
Revert the recent addition of memset()'s to TC_COMMIT. One of them is bogus and the other one needs more investigation to why valgrind is complaining.
Noticed and reverted by Phil Oester.
Diffstat (limited to 'libiptc')
-rw-r--r--libiptc/libiptc.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/libiptc/libiptc.c b/libiptc/libiptc.c
index 7bed221..d3d0fa7 100644
--- a/libiptc/libiptc.c
+++ b/libiptc/libiptc.c
@@ -2047,7 +2047,6 @@ TC_COMMIT(TC_HANDLE_T *handle)
counterlen = sizeof(STRUCT_COUNTERS_INFO)
+ sizeof(STRUCT_COUNTERS) * new_number;
- memset(repl, 0, sizeof(*repl) + (*handle)->entries->size);
/* These are the old counters we will get from kernel */
repl->counters = malloc(sizeof(STRUCT_COUNTERS)
@@ -2057,8 +2056,6 @@ TC_COMMIT(TC_HANDLE_T *handle)
errno = ENOMEM;
return 0;
}
- memset(repl->counters, 0, sizeof(STRUCT_COUNTERS)
- * (*handle)->info.num_entries);
/* These are the counters we're going to put back, later. */
newcounters = malloc(counterlen);
if (!newcounters) {