The flow-table module retrieves the destination and source
address as well as the transport protocol port information from the outbound packets. The routing code is generic and compares every byte in the given sockaddr object. Therefore the temporary sockaddr objects must be cleared due to padding bytes. In addition, the port information must be stripped or the route search will either fail or return the incorrect route entry. Unit testing is done using OpenVPN over the if_tun interface. MFC after: 7 days
This commit is contained in:
parent
63d46d1d5e
commit
688ba6823b
@ -593,6 +593,8 @@ flowtable_lookup_mbuf4(struct flowtable *ft, struct mbuf *m)
|
||||
|
||||
dsin = (struct sockaddr_in *)&dsa;
|
||||
ssin = (struct sockaddr_in *)&ssa;
|
||||
bzero(dsin, sizeof(*dsin));
|
||||
bzero(ssin, sizeof(*ssin));
|
||||
flags = ft->ft_flags;
|
||||
if (ipv4_mbuf_demarshal(ft, m, ssin, dsin, &flags) != 0)
|
||||
return (NULL);
|
||||
@ -796,6 +798,8 @@ flowtable_lookup_mbuf6(struct flowtable *ft, struct mbuf *m)
|
||||
|
||||
dsin6 = (struct sockaddr_in6 *)&dsa;
|
||||
ssin6 = (struct sockaddr_in6 *)&ssa;
|
||||
bzero(dsin6, sizeof(*dsin6));
|
||||
bzero(ssin6, sizeof(*ssin6));
|
||||
flags = ft->ft_flags;
|
||||
|
||||
if (ipv6_mbuf_demarshal(ft, m, ssin6, dsin6, &flags) != 0)
|
||||
@ -1088,6 +1092,14 @@ flowtable_lookup(struct flowtable *ft, struct sockaddr_storage *ssa,
|
||||
|
||||
ro = &sro;
|
||||
memcpy(&ro->ro_dst, dsa, sizeof(struct sockaddr_in));
|
||||
/*
|
||||
* The harvested source and destination addresses
|
||||
* may contain port information if the packet is
|
||||
* from a transport protocol (e.g. TCP/UDP). The
|
||||
* port field must be cleared before performing
|
||||
* a route lookup.
|
||||
*/
|
||||
((struct sockaddr_in *)&ro->ro_dst)->sin_port = 0;
|
||||
dsin = (struct sockaddr_in *)dsa;
|
||||
ssin = (struct sockaddr_in *)ssa;
|
||||
if ((dsin->sin_addr.s_addr == ssin->sin_addr.s_addr) ||
|
||||
@ -1105,6 +1117,7 @@ flowtable_lookup(struct flowtable *ft, struct sockaddr_storage *ssa,
|
||||
ro = (struct route *)&sro6;
|
||||
memcpy(&sro6.ro_dst, dsa,
|
||||
sizeof(struct sockaddr_in6));
|
||||
((struct sockaddr_in6 *)&ro->ro_dst)->sin6_port = 0;
|
||||
dsin6 = (struct sockaddr_in6 *)dsa;
|
||||
ssin6 = (struct sockaddr_in6 *)ssa;
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user