Re: [ovs-dev] [PATCH 1/2] ovn-controller: Fix memory leak and bug in setup_qos().

2017-02-02 Thread Ben Pfaff
On Thu, Feb 02, 2017 at 12:44:02PM +0530, Numan Siddique wrote: > On Wed, Feb 1, 2017 at 10:52 PM, Ben Pfaff wrote: > > > The caller of netdev_get_qos() is responsible for freeing its 'details' > > smap. > > > > > Is this patch a good candidate for back porting to 2.6 branch since

Re: [ovs-dev] [PATCH 1/2] ovn-controller: Fix memory leak and bug in setup_qos().

2017-02-01 Thread Numan Siddique
On Wed, Feb 1, 2017 at 10:52 PM, Ben Pfaff wrote: > The caller of netdev_get_qos() is responsible for freeing its 'details' > smap. > > Is this patch a good candidate for back porting to 2.6 branch since it's fixing memory leaks ? Thanks Numan ​ > Also, the previous qdisc

[ovs-dev] [PATCH 1/2] ovn-controller: Fix memory leak and bug in setup_qos().

2017-02-01 Thread Ben Pfaff
The caller of netdev_get_qos() is responsible for freeing its 'details' smap. Also, the previous qdisc details, for some arbitrary previous type of QoS, aren't relevant for changing the type of QoS. Signed-off-by: Ben Pfaff --- ovn/controller/binding.c | 7 +-- 1 file