aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPatrick McHardy <kaber@trash.net>2008-04-18 02:19:55 -0400
committerDavid S. Miller <davem@davemloft.net>2008-04-18 02:19:55 -0400
commitf5ba2d32170679eb9b7c251ac3d9687916a41c18 (patch)
treed0bd86075cf7437e9022955eb8813c388178f544
parent53083773dcbd3c80477e2ace143e361e1e806745 (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.c2
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);