summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTobias Klauser <tklauser@distanz.ch>2013-05-29 13:06:22 +0200
committerTobias Klauser <tklauser@distanz.ch>2013-05-29 13:06:22 +0200
commite9cb93501216c891347481a032ac058cc3deea2b (patch)
treefe37fe1e5992cad9488efe9a2c35a9545b400c72
parentf6c99a52f7714cae8cd048d758086e213e163571 (diff)
ifpps: Properly zero out allocated stats
The dynamically allocated members of struct ifstat have an entry for each CPU. Properly consider this when resetting the stats in the STATS_ZERO1 macro. Reported-by: Daniel Borkmann <borkmann@iogearbox.net> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
-rw-r--r--ifpps.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/ifpps.c b/ifpps.c
index 5e4b5b0..3dbb337 100644
--- a/ifpps.c
+++ b/ifpps.c
@@ -164,7 +164,7 @@ static void stats_alloc(struct ifstat *stats, int cpus)
}
#define STATS_ZERO1(member) \
- do { memset(stats->member, 0, sizeof(*(stats->member))); } while (0)
+ do { memset(stats->member, 0, cpus * sizeof(*(stats->member))); } while (0)
static void stats_zero(struct ifstat *stats, int cpus)
{