netflow: Correctly track flow creation time.
authorBen Pfaff <blp@nicira.com>
Thu, 10 Apr 2014 17:11:49 +0000 (10:11 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 10 Apr 2014 17:11:49 +0000 (10:11 -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>
Acked-by: Joe Stringer <joestringer@nicira.com>
ofproto/netflow.c

index f79ad6a..4c47e75 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;