secchan: Fix OpenFlow matching on output port with OFPFC_DELETE.

The implementation of matching on out_port was only half-implemented for
OFPFC_DELETE.  It was probably just overlooked.  This commit fixes it, by
supplying the other half.

diff --git a/secchan/ofproto.c b/secchan/ofproto.c
index b0cdec450e99c69c61537316aa64a3f3465945e3..0d8a6b8ba196f89c49bc0f81d9fb9bdadc0d0924 100644 (file)
--- a/secchan/ofproto.c
+++ b/secchan/ofproto.c
@@ -2556,6 +2556,11 @@ modify_flows_cb(struct cls_rule *rule_, void *cbdata_)
     struct rule *rule = rule_from_cls_rule(rule_);
     struct modify_flows_cbdata *cbdata = cbdata_;
 
+    if (cbdata->out_port != htons(OFPP_NONE)
+        && !rule_has_out_port(rule, cbdata->out_port)) {
+        return;
+    }
+
     modify_flow(cbdata->ofproto, cbdata->ofm, cbdata->n_actions,
                 cbdata->command, rule);
 }