The chassis_lookup_by_name function now calls sbrec_chassis_index_set_name instead of sbrec_chassis_set_name. Due to the use of the wrong API memory was leaked every time a chassis was looked up by name. This was mostly visible when chassis lookups had to be done continuously (e.g., when two chassis were misconfigured with the same system-id).
Reported-at: https://bugzilla.redhat.com/1698462 Reported-by: Alexander <[email protected]> Signed-off-by: Dumitru Ceara <[email protected]> --- ovn/lib/chassis-index.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ovn/lib/chassis-index.c b/ovn/lib/chassis-index.c index 34d4a31..423eeee 100644 --- a/ovn/lib/chassis-index.c +++ b/ovn/lib/chassis-index.c @@ -30,7 +30,7 @@ chassis_lookup_by_name(struct ovsdb_idl_index *sbrec_chassis_by_name, { struct sbrec_chassis *target = sbrec_chassis_index_init_row( sbrec_chassis_by_name); - sbrec_chassis_set_name(target, name); + sbrec_chassis_index_set_name(target, name); struct sbrec_chassis *retval = sbrec_chassis_index_find( sbrec_chassis_by_name, target); -- 1.8.3.1 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
