ofproto: Fix statistics of removed flow.

'fr' is a new variable on the stack. '+=' here adds the real statistics to a random stack memory. Fixes: 1644131 ("Add offload packets statistics") Acked-by: Roi Dayan <[email protected]> Signed-off-by: Ilya Maximets <[email protected]>
keyman9848 · May 15, 2020 · c36bba3 · c36bba3
1 parent 3c18bb0
commit c36bba3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
@@ -6085,8 +6085,8 @@ ofproto_rule_send_removed(struct rule *rule)
     fr.hard_timeout = rule->hard_timeout;
     ovs_mutex_unlock(&rule->mutex);
     rule->ofproto->ofproto_class->rule_get_stats(rule, &stats, &used);
-    fr.packet_count += stats.n_packets;
-    fr.byte_count += stats.n_bytes;
+    fr.packet_count = stats.n_packets;
+    fr.byte_count = stats.n_bytes;
     connmgr_send_flow_removed(connmgr, &fr);
     ovs_mutex_unlock(&ofproto_mutex);
 }