Commit 58075ff5 authored by YueHaibing's avatar YueHaibing Committed by David S. Miller

ipv4: fib_rules: Fix possible infinite loop in fib_empty_table

gcc warn this:
net/ipv4/fib_rules.c:203 fib_empty_table() warn:
 always true condition '(id <= 4294967295) => (0-u32max <= u32max)'

'id' is u32, which always not greater than RT_TABLE_MAX
(0xFFFFFFFF), So add a check to break while wrap around.
Signed-off-by: default avatarYueHaibing <yuehaibing@huawei.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 178fe944
...@@ -198,11 +198,15 @@ static int fib4_rule_match(struct fib_rule *rule, struct flowi *fl, int flags) ...@@ -198,11 +198,15 @@ static int fib4_rule_match(struct fib_rule *rule, struct flowi *fl, int flags)
static struct fib_table *fib_empty_table(struct net *net) static struct fib_table *fib_empty_table(struct net *net)
{ {
u32 id; u32 id = 1;
for (id = 1; id <= RT_TABLE_MAX; id++) while (1) {
if (!fib_get_table(net, id)) if (!fib_get_table(net, id))
return fib_new_table(net, id); return fib_new_table(net, id);
if (id++ == RT_TABLE_MAX)
break;
}
return NULL; return NULL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment