On Fri, May 10, 2019 at 10:58 AM Numan Siddique <[email protected]> wrote:
>
>
>
> On Thu, May 9, 2019 at 1:40 PM Dumitru Ceara <[email protected]> wrote:
>>
>> 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]>
>> ---
>
>
> Acked-by: Numan Siddique <[email protected]>
>
Hi,
Does this patch need more reviews in order to get committed?
Thanks,
Dumitru
>>
>> 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
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
