diff options
| author | Ben Pfaff <blp@nicira.com> | 2012-01-31 12:54:52 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2012-01-31 12:54:52 -0800 |
| commit | 30c42b95a67a00faf1b8734d5a9a0bcad6556d07 (patch) | |
| tree | 64aa120610998d22951dd21e95361c3c1c2c5607 | |
| parent | bd9d169bff274e5dd75d0d1d0d8e60c794a9fa86 (diff) | |
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 <mhu@nicira.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
| -rw-r--r-- | ofproto/ofproto-dpif.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index 9fb8fcca..b6e42f83 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) { |