Re: [ovs-dev] [PATCH v2 ovn] logical-fields: fix memory leak caused by initialize ovnfield_by_name twice

2020-03-10 Thread Damijan Skvarc
thanks for sharing your information with me. Obviously I didn't understand you well. thanks, damijan On Tue, 10 Mar 2020, 16:51 Numan Siddique, wrote: > On Tue, Mar 10, 2020 at 7:10 PM Damijan Skvarc > wrote: > > > > On Tue, Mar 10, 2020 at 2:02 PM Numan Siddique wrote: > > > > > On Sat, Mar 7

Re: [ovs-dev] [PATCH v2 ovn] logical-fields: fix memory leak caused by initialize ovnfield_by_name twice

2020-03-10 Thread Numan Siddique
On Tue, Mar 10, 2020 at 7:10 PM Damijan Skvarc wrote: > > On Tue, Mar 10, 2020 at 2:02 PM Numan Siddique wrote: > > > On Sat, Mar 7, 2020 at 4:42 AM Ben Pfaff wrote: > > > > > > On Thu, Mar 05, 2020 at 07:21:41AM +0100, Damijan Skvarc wrote: > > > > ovnfield_by_name is hash of strings which is u

Re: [ovs-dev] [PATCH v2 ovn] logical-fields: fix memory leak caused by initialize ovnfield_by_name twice

2020-03-10 Thread Damijan Skvarc
On Tue, Mar 10, 2020 at 2:02 PM Numan Siddique wrote: > On Sat, Mar 7, 2020 at 4:42 AM Ben Pfaff wrote: > > > > On Thu, Mar 05, 2020 at 07:21:41AM +0100, Damijan Skvarc wrote: > > > ovnfield_by_name is hash of strings which is used to quickly find > > > field by name. This hash is initialized fr

Re: [ovs-dev] [PATCH v2 ovn] logical-fields: fix memory leak caused by initialize ovnfield_by_name twice

2020-03-10 Thread Numan Siddique
On Sat, Mar 7, 2020 at 4:42 AM Ben Pfaff wrote: > > On Thu, Mar 05, 2020 at 07:21:41AM +0100, Damijan Skvarc wrote: > > ovnfield_by_name is hash of strings which is used to quickly find > > field by name. This hash is initialized from ovn_init_symtab(). In case > > the latter function is called mu

Re: [ovs-dev] [PATCH v2 ovn] logical-fields: fix memory leak caused by initialize ovnfield_by_name twice

2020-03-06 Thread Ben Pfaff
On Thu, Mar 05, 2020 at 07:21:41AM +0100, Damijan Skvarc wrote: > ovnfield_by_name is hash of strings which is used to quickly find > field by name. This hash is initialized from ovn_init_symtab(). In case > the latter function is called multiple times then also ovnfield_by_name is > initialized mu

[ovs-dev] [PATCH v2 ovn] logical-fields: fix memory leak caused by initialize ovnfield_by_name twice

2020-03-04 Thread Damijan Skvarc
ovnfield_by_name is hash of strings which is used to quickly find field by name. This hash is initialized from ovn_init_symtab(). In case the latter function is called multiple times then also ovnfield_by_name is initialized multiple times but without freeing previously allocated memory resources w