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 | |
| parent | fcdf95cb293603acdff910715c8aa2b19ed29df4 (diff) | |
| download | linux-3a53943b5ae8b61913e2d61e98cbeedf67861c92.tar.bz2 | |
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.c | 2 |
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)) |