From: Paul Blakey <pa...@mellanox.com>
Travis complains about variable length array usage, use
a fixed size instead.
Signed-off-by: Paul Blakey <pa...@mellanox.com>
Reviewed-by: Roi Dayan <r...@mellanox.com>
---
lib/netdev-tc-offloads.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c
index 781d849..6ad2551 100644
--- a/lib/netdev-tc-offloads.c
+++ b/lib/netdev-tc-offloads.c
@@ -581,13 +581,15 @@ parse_put_flow_set_masked_action(struct tc_flower *flower,
bool hasmask)
{
static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 20);
- char *set_buff[set_len], *set_data, *set_mask;
+ char *set_buff[128], *set_data, *set_mask;
char *key = (char *) &flower->rewrite.key;
char *mask = (char *) &flower->rewrite.mask;
const struct nlattr *attr;
int i, j, type;
size_t size;
+ ovs_assert(set_len <= 128);
+
/* copy so we can set attr mask to 0 for used ovs key struct members */
memcpy(set_buff, set, set_len);
attr = (struct nlattr *) set_buff;
--
2.7.5
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
