[ovs-dev,03/10] ovn-northd: Fix memory leak in ARP reply flows.
diff mbox

Message ID 1445412271-22019-3-git-send-email-jpettit@nicira.com
State Deferred
Headers show

Commit Message

Justin Pettit Oct. 21, 2015, 7:24 a.m. UTC
Signed-off-by: Justin Pettit <jpettit@nicira.com>
---
 ovn/northd/ovn-northd.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Comments

Ben Pfaff Oct. 22, 2015, 11:01 p.m. UTC | #1
On Wed, Oct 21, 2015 at 12:24:24AM -0700, Justin Pettit wrote:
> Signed-off-by: Justin Pettit <jpettit@nicira.com>

Acked-by: Ben Pfaff <blp@nicira.com>
Justin Pettit Oct. 22, 2015, 11:03 p.m. UTC | #2
> On Oct 22, 2015, at 4:01 PM, Ben Pfaff <blp@nicira.com> wrote:
> 
> On Wed, Oct 21, 2015 at 12:24:24AM -0700, Justin Pettit wrote:
>> Signed-off-by: Justin Pettit <jpettit@nicira.com>
> 
> Acked-by: Ben Pfaff <blp@nicira.com>

Thanks. I pushed that to master.

--Justin

Patch
diff mbox

diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c
index e199937..2ebd3f4 100644
--- a/ovn/northd/ovn-northd.c
+++ b/ovn/northd/ovn-northd.c
@@ -1405,6 +1405,8 @@  build_lrouter_flows(struct hmap *datapaths, struct hmap *ports,
             op->json_key);
         ovn_lflow_add(lflows, op->od, S_ROUTER_IN_IP_INPUT, 90,
                       match, actions);
+        free(match);
+        free(actions);
 
         /* Drop IP traffic to this router. */
         match = xasprintf("ip4.dst == "IP_FMT, IP_ARGS(op->ip));