From: Ben Pfaff <blp@nicira.com>
Date: Mon, 17 Mar 2014 21:32:16 +0000 (-0700)
Subject: netflow: Correctly track flow creation time.
X-Git-Tag: sliver-openvswitch-2.2.90-1~3^2~147
X-Git-Url: http://git.onelab.eu/?p=sliver-openvswitch.git;a=commitdiff_plain;h=e1f208cc4503bad45c1d8edc8bfed5fb433dd02f

netflow: Correctly track flow creation time.

'created' is supposed to be the time the flow was created, but it was
getting reset to zero on every expiration, causing the flow start time to
be wonky after the first active expiration on a flow.

Reported-by: Lior Neudorfer <lior@guardicore.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
---

diff --git a/ofproto/netflow.c b/ofproto/netflow.c
index 7729906c1..e9382afe9 100644
--- a/ofproto/netflow.c
+++ b/ofproto/netflow.c
@@ -270,7 +270,6 @@ netflow_expire__(struct netflow *nf, struct netflow_flow *nf_flow)
     }
 
     /* Update flow tracking data. */
-    nf_flow->created = 0;
     nf_flow->packet_count = 0;
     nf_flow->byte_count = 0;
     nf_flow->tcp_flags = 0;