diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-12-14 02:30:00 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-12-14 13:34:55 -0500 |
commit | 3a53943b5ae8b61913e2d61e98cbeedf67861c92 (patch) | |
tree | a3edab0ac31a74f722e066529df76b03ef8b96f7 /net/sched/cls_flow.c | |
parent | fcdf95cb293603acdff910715c8aa2b19ed29df4 (diff) | |
download | op-kernel-dev-3a53943b5ae8b61913e2d61e98cbeedf67861c92.zip op-kernel-dev-3a53943b5ae8b61913e2d61e98cbeedf67861c92.tar.gz |
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>
Diffstat (limited to 'net/sched/cls_flow.c')
-rw-r--r-- | net/sched/cls_flow.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sched/cls_flow.c b/net/sched/cls_flow.c index 51ff194..1d8bd0d 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)) |