diff options
author | Thomas Graf <tgraf@suug.ch> | 2006-11-10 17:11:04 -0500 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2006-12-03 00:22:12 -0500 |
commit | f465e489c457821751f6253abaaf28c10f9d8895 (patch) | |
tree | d36ceef24267d7b5fe4d836435a5760dc64f67be /net/ipv6/fib6_rules.c | |
parent | 339bf98ffc6a8d8eb16fc532ac57ffbced2f8a68 (diff) |
[IPv6] rules: Remove bogus tos validation check
Noticed by Al Viro:
(frh->tos & ~IPV6_FLOWINFO_MASK))
where IPV6_FLOWINFO_MASK is htonl(0xfffffff) and frh->tos
is u8, which makes no sense here...
Signed-off-by: Thomas Graf <tgraf@suug.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/fib6_rules.c')
-rw-r--r-- | net/ipv6/fib6_rules.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/ipv6/fib6_rules.c b/net/ipv6/fib6_rules.c index d587dde5897e..0862809ffcf7 100644 --- a/net/ipv6/fib6_rules.c +++ b/net/ipv6/fib6_rules.c | |||
@@ -142,8 +142,7 @@ static int fib6_rule_configure(struct fib_rule *rule, struct sk_buff *skb, | |||
142 | int err = -EINVAL; | 142 | int err = -EINVAL; |
143 | struct fib6_rule *rule6 = (struct fib6_rule *) rule; | 143 | struct fib6_rule *rule6 = (struct fib6_rule *) rule; |
144 | 144 | ||
145 | if (frh->src_len > 128 || frh->dst_len > 128 || | 145 | if (frh->src_len > 128 || frh->dst_len > 128) |
146 | (frh->tos & ~IPV6_FLOWINFO_MASK)) | ||
147 | goto errout; | 146 | goto errout; |
148 | 147 | ||
149 | if (rule->action == FR_ACT_TO_TBL) { | 148 | if (rule->action == FR_ACT_TO_TBL) { |