netflow: Correctly track flow creation time.
authorBen Pfaff <blp@nicira.com>
Mon, 17 Mar 2014 21:32:16 +0000 (14:32 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 10 Apr 2014 17:06:12 +0000 (10:06 -0700)
'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>
ofproto/netflow.c

index 7729906..e9382af 100644 (file)
@@ -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;