ipvs: Explictly clear ip_vs_stats members
authorSimon Horman <horms@verge.net.au>
Mon, 11 Aug 2008 07:19:14 +0000 (17:19 +1000)
committerSven Wegener <sven.wegener@stealer.net>
Mon, 11 Aug 2008 12:00:55 +0000 (14:00 +0200)
In order to align the coding styles of ip_vs_zero_stats() and
its child-function ip_vs_zero_estimator(), clear ip_vs_stats
members explicitlty rather than doing a limited memset().

This was chosen over modifying ip_vs_zero_estimator() to use
memset() as it is more robust against changes in members
in the relevant structures. memset() would be prefered if
all members of the structure were to be cleared.

Cc: Sven Wegener <sven.wegener@stealer.net>
Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: Sven Wegener <sven.wegener@stealer.net>
net/ipv4/ipvs/ip_vs_ctl.c

index cfb1d20993d1d412042a61501d73b57c27f3d8af..6379705a8dcb2a98505af97ee4fa0f23045d61f8 100644 (file)
@@ -683,8 +683,21 @@ static void
 ip_vs_zero_stats(struct ip_vs_stats *stats)
 {
        spin_lock_bh(&stats->lock);
-       memset(stats, 0, (char *)&stats->lock - (char *)stats);
+
+       stats->conns = 0;
+       stats->inpkts = 0;
+       stats->outpkts = 0;
+       stats->inbytes = 0;
+       stats->outbytes = 0;
+
+       stats->cps = 0;
+       stats->inpps = 0;
+       stats->outpps = 0;
+       stats->inbps = 0;
+       stats->outbps = 0;
+
        ip_vs_zero_estimator(stats);
+
        spin_unlock_bh(&stats->lock);
 }