summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Dumazet <eric.dumazet@gmail.com>2011-12-14 02:30:00 +0000
committerDavid S. Miller <davem@davemloft.net>2011-12-14 13:34:55 -0500
commit3a53943b5ae8b61913e2d61e98cbeedf67861c92 (patch)
treea3edab0ac31a74f722e066529df76b03ef8b96f7
parentfcdf95cb293603acdff910715c8aa2b19ed29df4 (diff)
downloadlinux-stable-3a53943b5ae8b61913e2d61e98cbeedf67861c92.tar.gz
linux-stable-3a53943b5ae8b61913e2d61e98cbeedf67861c92.tar.bz2
linux-stable-3a53943b5ae8b61913e2d61e98cbeedf67861c92.zip
cls_flow: remove one dynamic array
Its better to use a predefined size for this small automatic variable. Removes a sparse error as well : net/sched/cls_flow.c:288:13: error: bad constant expression Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/sched/cls_flow.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sched/cls_flow.c b/net/sched/cls_flow.c
index 51ff19485e12..1d8bd0dbcd1f 100644
--- a/net/sched/cls_flow.c
+++ b/net/sched/cls_flow.c
@@ -285,7 +285,7 @@ static int flow_classify(struct sk_buff *skb, const struct tcf_proto *tp,
int r;
list_for_each_entry(f, &head->filters, list) {
- u32 keys[f->nkeys];
+ u32 keys[FLOW_KEY_MAX + 1];
struct flow_keys flow_keys;
if (!tcf_em_tree_match(skb, &f->ematches, NULL))