diff options
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 |
commit | 6cf0411600b405b71dcff976670df8f8d2fe8622 (patch) | |
tree | fd866fa0a50cc6a0141ed4aea3f2b7db3b686b32 /libiptc/libiptc.c | |
parent | 5244bd949f12e7c1dba979fad52bcb5ad6dca09a (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/libiptc.c')
-rw-r--r-- | libiptc/libiptc.c | 3 |
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) { |