diff mbox

[ovs-dev,1/3] dpif-netlink: Fix memory leak on port del on WIN32.

Message ID 20170412201948.30978-1-joe@ovn.org
State Accepted
Headers show

Commit Message

Joe Stringer April 12, 2017, 8:19 p.m. UTC 
From: Eric Garver <e@erig.me>

Fixes: da467899ab6e ("Windows: Add internal switch port per OVS bridge")
Signed-off-by: Eric Garver <e@erig.me>
Acked-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Signed-off-by: Joe Stringer <joe@ovn.org>
---
 lib/dpif-netlink.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Joe Stringer April 12, 2017, 8:59 p.m. UTC | #1 
On 12 April 2017 at 13:19, Joe Stringer <joe@ovn.org> wrote:
> From: Eric Garver <e@erig.me>
>
> Fixes: da467899ab6e ("Windows: Add internal switch port per OVS bridge")
> Signed-off-by: Eric Garver <e@erig.me>
> Acked-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
> Signed-off-by: Joe Stringer <joe@ovn.org>
> ---

Applied to master and branch-2.7.
Alin Serdean April 12, 2017, 11:18 p.m. UTC | #2 
Thanks!

> -----Original Message-----
> From: ovs-dev-bounces@openvswitch.org [mailto:ovs-dev-
> bounces@openvswitch.org] On Behalf Of Joe Stringer
> Sent: Thursday, April 13, 2017 12:00 AM
> To: ovs dev <dev@openvswitch.org>
> Cc: Eric Garver <e@erig.me>
> Subject: Re: [ovs-dev] [PATCH 1/3] dpif-netlink: Fix memory leak on port del
> on WIN32.
> 
> On 12 April 2017 at 13:19, Joe Stringer <joe@ovn.org> wrote:
> > From: Eric Garver <e@erig.me>
> >
> > Fixes: da467899ab6e ("Windows: Add internal switch port per OVS bridge")
> > Signed-off-by: Eric Garver <e@erig.me>
> > Acked-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
> > Signed-off-by: Joe Stringer <joe@ovn.org>
> > ---
> 
> Applied to master and branch-2.7.
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
diff mbox

Patch

diff --git a/lib/dpif-netlink.c b/lib/dpif-netlink.c
index bad575c9cf65..e0acf5cce3cf 100644
--- a/lib/dpif-netlink.c
+++ b/lib/dpif-netlink.c
@@ -963,6 +963,7 @@  dpif_netlink_port_del__(struct dpif_netlink *dpif, odp_port_t port_no)
                      temp_dpif_port.name);
         };
     }
+    dpif_port_destroy(&temp_dpif_port);
 #endif
     error = dpif_netlink_vport_transact(&vport, NULL, NULL);