Re: [ovs-dev] [PATCH ovn] northd: Fix memory leak.

2022-08-17 Thread Mark Michelson
Thanks Olaf and Ales, I have merged this to main, branch-22.06, and branch-22.03. On 8/17/22 03:59, Ales Musil wrote: On Wed, Aug 17, 2022 at 9:59 AM Olaf Seibert via dev < ovs-dev@openvswitch.org> wrote: Commit 7b56f69580e1f390d9c6753a2cb8f0dbfbb4c467 cloned the external_ids to ids, then pas

Re: [ovs-dev] [PATCH ovn] northd: Fix memory leak.

2022-08-17 Thread Ales Musil
On Wed, Aug 17, 2022 at 9:59 AM Olaf Seibert via dev < ovs-dev@openvswitch.org> wrote: > Commit 7b56f69580e1f390d9c6753a2cb8f0dbfbb4c467 cloned the external_ids > to ids, then passed them to sbrec_port_binding_set_external_ids(). > The intermediate step is unneeded (and caused a memory leak). > Pa

[ovs-dev] [PATCH ovn] northd: Fix memory leak.

2022-08-17 Thread Olaf Seibert via dev
Commit 7b56f69580e1f390d9c6753a2cb8f0dbfbb4c467 cloned the external_ids to ids, then passed them to sbrec_port_binding_set_external_ids(). The intermediate step is unneeded (and caused a memory leak). Pass the external_ids directly. Signed-off-by: Olaf Seibert --- northd/northd.c | 4 +--- 1 fil