diff mbox series

[ovs-dev,1/7] netdev-tc-offloads: Fix travis compilation error

Message ID 1511268042-65124-2-git-send-email-roid@mellanox.com
State Accepted
Headers show
Series Fixes for header rewrite feature | expand

Commit Message

Roi Dayan Nov. 21, 2017, 12:40 p.m. UTC 
From: Paul Blakey <paulb@mellanox.com>

Travis complains about variable length array usage, use
a fixed size instead.

Signed-off-by: Paul Blakey <paulb@mellanox.com>
Reviewed-by: Roi Dayan <roid@mellanox.com>
---
 lib/netdev-tc-offloads.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff mbox series

Patch

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;