diff options
author | Patrick McHardy <kaber@trash.net> | 2008-04-18 02:19:55 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-04-18 02:19:55 -0400 |
commit | f5ba2d32170679eb9b7c251ac3d9687916a41c18 (patch) | |
tree | d0bd86075cf7437e9022955eb8813c388178f544 | |
parent | 53083773dcbd3c80477e2ace143e361e1e806745 (diff) |
[PKT_SCHED]: Fix datalen check in tcf_simp_init().
datalen is unsigned so it can never be less than zero,
but that's ok because the attribute passed to nla_len()
has been validated and therefore a negative return
value is impossible.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/sched/act_simple.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sched/act_simple.c b/net/sched/act_simple.c index fbde461b716c..64b2d136c78e 100644 --- a/net/sched/act_simple.c +++ b/net/sched/act_simple.c | |||
@@ -115,7 +115,7 @@ static int tcf_simp_init(struct nlattr *nla, struct nlattr *est, | |||
115 | return -EINVAL; | 115 | return -EINVAL; |
116 | 116 | ||
117 | datalen = nla_len(tb[TCA_DEF_DATA]); | 117 | datalen = nla_len(tb[TCA_DEF_DATA]); |
118 | if (datalen <= 0) | 118 | if (datalen == 0) |
119 | return -EINVAL; | 119 | return -EINVAL; |
120 | 120 | ||
121 | pc = tcf_hash_check(parm->index, a, bind, &simp_hash_info); | 121 | pc = tcf_hash_check(parm->index, a, bind, &simp_hash_info); |