diff options
| author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-12-13 21:30:00 -0500 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-12-14 13:34:55 -0500 |
| commit | 3a53943b5ae8b61913e2d61e98cbeedf67861c92 (patch) | |
| tree | a3edab0ac31a74f722e066529df76b03ef8b96f7 | |
| parent | fcdf95cb293603acdff910715c8aa2b19ed29df4 (diff) | |
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, | |||
| 285 | int r; | 285 | int r; |
| 286 | 286 | ||
| 287 | list_for_each_entry(f, &head->filters, list) { | 287 | list_for_each_entry(f, &head->filters, list) { |
| 288 | u32 keys[f->nkeys]; | 288 | u32 keys[FLOW_KEY_MAX + 1]; |
| 289 | struct flow_keys flow_keys; | 289 | struct flow_keys flow_keys; |
| 290 | 290 | ||
| 291 | if (!tcf_em_tree_match(skb, &f->ematches, NULL)) | 291 | if (!tcf_em_tree_match(skb, &f->ematches, NULL)) |