From 30c42b95a67a00faf1b8734d5a9a0bcad6556d07 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Tue, 31 Jan 2012 12:54:52 -0800 Subject: [PATCH] ofproto-dpif: Fix merge error that caused memory leak. Commit 5aa0fe5be (ofproto-dpif: Fix use-after-free error in handle_miss_upcalls().), a crossport from master of commit 33bb0caa6, did not include this necessary change, causing a memory leak. Bug #9493. Reported-by: Michael Hu Signed-off-by: Ben Pfaff --- ofproto/ofproto-dpif.c | 1 - 1 file changed, 1 deletion(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index 9fb8fccac..b6e42f831 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -2537,7 +2537,6 @@ handle_flow_miss(struct ofproto_dpif *ofproto, struct flow_miss *miss, struct flow_miss_op *op; struct dpif_execute *execute; - list_remove(&packet->list_node); ofproto->n_matches++; if (facet->rule->up.cr.priority == FAIL_OPEN_PRIORITY) { -- 2.43.0