d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Correct warnings comparing unsigned variables < 0 constantly reported

Browse Source 
while building kernels.  All instances removed are indeed unsigned so
the expressions could not be true.

MFC after:	1 week
This commit is contained in:
Bjoern A. Zeeb 2013-12-25 20:08:44 +00:00
parent 6a4d909530
commit 010c2b8192
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
sys/netinet6/in6_mcast.c
View File

@ -1851,8 +1851,7 @@ in6p_join_group(struct inpcb *inp, struct sockopt *sopt)
if (mreq.ipv6mr_interface == 0) {
ifp = in6p_lookup_mcast_ifp(inp, &gsa->sin6);
} else {
if (mreq.ipv6mr_interface < 0 ||
V_if_index < mreq.ipv6mr_interface)
if (V_if_index < mreq.ipv6mr_interface)
return (EADDRNOTAVAIL);
ifp = ifnet_byindex(mreq.ipv6mr_interface);
}
@ -2198,7 +2197,7 @@ in6p_leave_group(struct inpcb *inp, struct sockopt *sopt)
* XXX SCOPE6 lock potentially taken here.
*/
if (ifindex != 0) {
if (ifindex < 0 || V_if_index < ifindex)
if (V_if_index < ifindex)
return (EADDRNOTAVAIL);
ifp = ifnet_byindex(ifindex);
if (ifp == NULL)
@ -2356,7 +2355,7 @@ in6p_set_multicast_if(struct inpcb *inp, struct sockopt *sopt)
error = sooptcopyin(sopt, &ifindex, sizeof(u_int), sizeof(u_int));
if (error)
return (error);
if (ifindex < 0 || V_if_index < ifindex)
if (V_if_index < ifindex)
return (EINVAL);
ifp = ifnet_byindex(ifindex);