From b969b51164f028a728efded35550a02926b5aa20 Mon Sep 17 00:00:00 2001 From: Justin Pettit Date: Sat, 25 Oct 2008 13:29:17 -0700 Subject: [PATCH] Fix leaking of flows when output action validation fails. Thanks to Brandon Heller for catching this. --- switch/datapath.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/switch/datapath.c b/switch/datapath.c index fb61d0aeb..8fdc9d3a6 100644 --- a/switch/datapath.c +++ b/switch/datapath.c @@ -992,7 +992,7 @@ add_flow(struct datapath *dp, const struct sender *sender, if (v_code != ACT_VALIDATION_OK) { dp_send_error_msg(dp, sender, OFPET_BAD_ACTION, v_code, ofm, ntohs(ofm->header.length)); - goto error; + goto error_free_flow; } /* Fill out flow. */ -- 2.43.0