diff mbox

[ovs-dev,v1] ovn-northd: Fix {}-enclosed constants for ND responder

Message ID 1469512946-19732-1-git-send-email-zealokii@gmail.com
State Accepted
Headers show

Commit Message

Zong Kai LI July 26, 2016, 6:02 a.m. UTC
It missed comma as constant seperator in match string for ND responder.

Signed-off-by: Zong Kai LI <zealokii@gmail.com>
---
 ovn/northd/ovn-northd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Russell Bryant July 26, 2016, 9:06 p.m. UTC | #1
On Tue, Jul 26, 2016 at 2:02 AM, Zong Kai LI <zealokii@gmail.com> wrote:

> It missed comma as constant seperator in match string for ND responder.
>
> Signed-off-by: Zong Kai LI <zealokii@gmail.com>


Thanks!  I applied this to master.
diff mbox

Patch

diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c
index a3d1672..8e5c5c5 100644
--- a/ovn/northd/ovn-northd.c
+++ b/ovn/northd/ovn-northd.c
@@ -2018,7 +2018,7 @@  build_lswitch_flows(struct hmap *datapaths, struct hmap *ports,
                 } else {
                     ds_put_format(&match, "nd.target == {");
                     for (size_t j = 0; j < op->lsp_addrs[i].n_ipv6_addrs; j++) {
-                        ds_put_cstr(&match,
+                        ds_put_format(&match, "%s, ",
                                       op->lsp_addrs[i].ipv6_addrs[j].addr_s);
                     }
                     ds_chomp(&match, ' ');