diff options
author | Lorenzo Bianconi <lorenzo.bianconi@redhat.com> | 2018-01-05 18:52:00 +0100 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2018-01-08 08:41:41 -0800 |
commit | eddb84a1646c07c7c8ca5f5f147f4231273c235e (patch) | |
tree | 65cde90ba6068e91ff42cc530ce3e94d2b3a7962 /ovn/northd | |
parent | 17551f585b4e8e6d806c117cd69f4f2031969d26 (diff) | |
download | openvswitch-eddb84a1646c07c7c8ca5f5f147f4231273c235e.tar.gz |
OVN: remove useless ds_clear() on actions ds
Remove ds_clear() on actions dynamic string in build_acls()
since they have just been initialized to DS_EMPTY_INITIALIZER
Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'ovn/northd')
-rw-r--r-- | ovn/northd/ovn-northd.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index e3ddc1fd9..63ed97efb 100644 --- a/ovn/northd/ovn-northd.c +++ b/ovn/northd/ovn-northd.c @@ -3272,7 +3272,6 @@ build_acls(struct ovn_datapath *od, struct hmap *lflows) "(!ct.est || (ct.est && ct_label.blocked == 1)) " "&& (%s)", acl->match); - ds_clear(&actions); build_acl_log(&actions, acl); ds_put_cstr(&actions, "/* drop */"); ovn_lflow_add_with_hint(lflows, od, stage, @@ -3307,7 +3306,6 @@ build_acls(struct ovn_datapath *od, struct hmap *lflows) /* There are no stateful ACLs in use on this datapath, * so a "drop" ACL is simply the "drop" logical flow action * in all cases. */ - ds_clear(&actions); build_acl_log(&actions, acl); ds_put_cstr(&actions, "/* drop */"); ovn_lflow_add_with_hint(lflows, od, stage, |