From b6f67919d48078d2171927b291d0a2ac629c4535 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Fri, 28 Aug 2009 14:59:42 -0700 Subject: [PATCH] secchan: Avoid sending NetFlow packets for empty flows. There is no value in sending out NetFlow messages when the byte counter (hence, packet counter) is 0. This does not often happen, but it can in corner cases where a flow gets installed but never sees any traffic before it is uninstalled. CC: Peter Balland --- secchan/ofproto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/secchan/ofproto.c b/secchan/ofproto.c index efa5c9b9e..b3fef1b71 100644 --- a/secchan/ofproto.c +++ b/secchan/ofproto.c @@ -1696,7 +1696,7 @@ rule_post_uninstall(struct ofproto *ofproto, struct rule *rule) struct rule *super = rule->super; rule_account(ofproto, rule, 0); - if (ofproto->netflow) { + if (ofproto->netflow && rule->byte_count) { struct ofexpired expired; expired.flow = rule->cr.flow; expired.packet_count = rule->packet_count; -- 2.43.0