secchan: Optimize no-change case in modify_flow().

diff --git a/secchan/ofproto.c b/secchan/ofproto.c
index 5b7e169f1a89f41d327a30554b472aa27829bea2..8b194986481b0c311b96a451d08c646fc7cbdf7f 100644 (file)
--- a/secchan/ofproto.c
+++ b/secchan/ofproto.c
@@ -2497,9 +2497,16 @@ modify_flow(struct ofproto *p, const struct ofp_flow_mod *ofm,
     if (command == OFPFC_DELETE) {
         rule_remove(p, rule);
     } else {
+        size_t actions_len = n_actions * sizeof *rule->actions;
+
+        if (n_actions == rule->n_actions
+            && !memcmp(ofm->actions, rule->actions, actions_len))
+        {
+            return 0;
+        }
+
         free(rule->actions);
-        rule->actions = xmemdup(ofm->actions,
-                                n_actions * sizeof *rule->actions);
+        rule->actions = xmemdup(ofm->actions, actions_len);
         rule->n_actions = n_actions;
 
         if (rule->cr.wc.wildcards) {