On 01/22/2018 09:50 AM, Ilya Maximets wrote:
The caller takes ownership over allocated array of counters.
And it must free them.
CC: Michal Weglicki <[email protected]>
Fixes: 971f4b394c6e ("netdev: Custom statistics.")
Signed-off-by: Ilya Maximets <[email protected]>
---
vswitchd/bridge.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index d80da1c..02d97de 100644
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -2432,6 +2432,7 @@ iface_refresh_stats(struct iface *iface)
free(values);
free(keys);
+ free(custom_stats.counters);
The sentiment is correct here, but I believe that rather than freeing
custom_stats.counters directly, you should call
netdev_free_custom_stats_counters(&custom_stats);
}
static void
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
