diff options
author | David S. Miller <davem@davemloft.net> | 2015-01-18 23:36:08 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-01-18 23:36:08 -0500 |
commit | 7b46a644a407f9f2f1c8f7b2af157c79af55b49e (patch) | |
tree | 5db384821399a6e07abd70a61cc0afd454144e27 /net/ipv6 | |
parent | 27c00132852b77b12e625cd6a0ccf43d6bf5795f (diff) |
netlink: Fix bugs in nlmsg_end() conversions.
Commit 053c095a82cf ("netlink: make nlmsg_end() and genlmsg_end()
void") didn't catch all of the cases where callers were breaking out
on the return value being equal to zero, which they no longer should
when zero means success.
Fix all such cases.
Reported-by: Marcel Holtmann <marcel@holtmann.org>
Reported-by: Scott Feldman <sfeldma@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6')
-rw-r--r-- | net/ipv6/addrconf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c index 8975d9501d50..d6b4f5d08014 100644 --- a/net/ipv6/addrconf.c +++ b/net/ipv6/addrconf.c | |||
@@ -4213,7 +4213,7 @@ static int inet6_dump_addr(struct sk_buff *skb, struct netlink_callback *cb, | |||
4213 | goto cont; | 4213 | goto cont; |
4214 | 4214 | ||
4215 | if (in6_dump_addrs(idev, skb, cb, type, | 4215 | if (in6_dump_addrs(idev, skb, cb, type, |
4216 | s_ip_idx, &ip_idx) <= 0) | 4216 | s_ip_idx, &ip_idx) < 0) |
4217 | goto done; | 4217 | goto done; |
4218 | cont: | 4218 | cont: |
4219 | idx++; | 4219 | idx++; |