Martin Pieuchot <mpieuc...@nolizard.org> writes: > Diff below replace in_iawithaddr() + in_broadcast() -> ifa_ifwithaddr(), > that does the same for IPv4 since broadcast addresses are added to the > tree.
This prevents listeners to bind on 255.255.255.255, something allowed with the current code. Thoughts? > ok? > > Index: netinet/raw_ip.c > =================================================================== > RCS file: /home/ncvs/src/sys/netinet/raw_ip.c,v > retrieving revision 1.72 > diff -u -p -r1.72 raw_ip.c > --- netinet/raw_ip.c 21 Apr 2014 12:22:26 -0000 1.72 > +++ netinet/raw_ip.c 27 May 2014 11:02:45 -0000 > @@ -465,9 +465,8 @@ rip_usrreq(struct socket *so, int req, s > break; > } > if (!((so->so_options & SO_BINDANY) || > - addr->sin_addr.s_addr == 0 || > - in_iawithaddr(addr->sin_addr, inp->inp_rtableid) || > - in_broadcast(addr->sin_addr, NULL, inp->inp_rtableid))) { > + addr->sin_addr.s_addr == INADDR_ANY || > + ifa_ifwithaddr(sintosa(addr), inp->inp_rtableid))) { > error = EADDRNOTAVAIL; > break; > } > -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE