When a route is withdrawn (blackholed) the netlink message doesn't include
an RTA_OIF element. This results in an "unexpected netlink message
contents" log message because this element is not optional.
Given that the netlink message will be ignored anyway, and subsequent
error checking will cope with missing RTA_OIF, the element should be
optional in order to suppress unnecessary log messages.
Signed-off-by: Tony van der Peet <tony.vanderp...@alliedtelesis.co.nz>
---
lib/route-table.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/route-table.c b/lib/route-table.c
index 6a3eeb504..d1c71def2 100644
--- a/lib/route-table.c
+++ b/lib/route-table.c
@@ -188,7 +188,7 @@ route_table_parse(struct ofpbuf *buf, struct
route_table_msg *change)
static const struct nl_policy policy[] = {
[RTA_DST] = { .type = NL_A_U32, .optional = true },
- [RTA_OIF] = { .type = NL_A_U32, .optional = false },
+ [RTA_OIF] = { .type = NL_A_U32, .optional = true },
[RTA_GATEWAY] = { .type = NL_A_U32, .optional = true },
};
--
2.11.0
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
