mirrors/passt
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

icmp: Don't attempt to handle "wrong direction" ping socket traffic

Browse source 
Linux ICMP "ping" sockets are very specific in what they do.  They let
userspace send ping requests (ICMP_ECHO or ICMP6_ECHO_REQUEST), and receive
matching replies (ICMP_ECHOREPLY or ICMP6_ECHO_REPLY).  They don't let you
intercept or handle incoming ping requests.

In the case of passt/pasta that means we can process echo requests from tap
and forward them to a ping socket, then take the replies from the ping
socket and forward them to tap.  We can't do the reverse: take echo
requests from the host and somehow forward them to the guest. There's
really no way for something outside to initiate a ping to a passt/pasta
connected guest and if there was we'd need an entirely different mechanism
to handle it.

However, we have some logic to deal with packets going in that reverse
direction.  Remove it, since it can't ever be used that way.  While we're
there use defines for the ICMPv6 types, instead of open coded type values.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
This commit is contained in:
David Gibson 2024-01-16 16:16:10 +11:00 committed by Stefano Brivio
parent 2cb2fe6f89
commit 8534cdbfd1
1 changed files with 6 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
icmp.c
View file

@ -93,8 +93,7 @@ void icmp_sock_handler(const struct ctx *c, union epoll_ref ref)
icmp_id_map[V4][id].seq = seq; icmp_id_map[V4][id].seq = seq;
} }
debug("ICMP: echo %s to tap, ID: %i, seq: %i", debug("ICMP: echo reply to tap, ID: %i, seq: %i", id, seq);
(ih->type == ICMP_ECHO) ? "request" : "reply", id, seq);
tap_icmp4_send(c, sr.sin_addr, tap_ip4_daddr(c), buf, n); tap_icmp4_send(c, sr.sin_addr, tap_ip4_daddr(c), buf, n);
} }
@ -138,8 +137,7 @@ void icmpv6_sock_handler(const struct ctx *c, union epoll_ref ref)
icmp_id_map[V6][id].seq = seq; icmp_id_map[V6][id].seq = seq;
} }
debug("ICMPv6: echo %s to tap, ID: %i, seq: %i", debug("ICMPv6: echo reply to tap, ID: %i, seq: %i", id, seq);
(ih->icmp6_type == 128) ? "request" : "reply", id, seq);
tap_icmp6_send(c, &sr.sin6_addr, tap_icmp6_send(c, &sr.sin6_addr,
tap_ip6_daddr(c, &sr.sin6_addr), buf, n); tap_ip6_daddr(c, &sr.sin6_addr), buf, n);
@ -178,7 +176,7 @@ int icmp_tap_handler(const struct ctx *c, uint8_t pif, int af,
if (!ih) if (!ih)
return 1; return 1;
if (ih->type != ICMP_ECHO && ih->type != ICMP_ECHOREPLY) if (ih->type != ICMP_ECHO)
return 1; return 1;
iref.id = id = ntohs(ih->un.echo.id); iref.id = id = ntohs(ih->un.echo.id);
@ -205,8 +203,7 @@ int icmp_tap_handler(const struct ctx *c, uint8_t pif, int af,
(struct sockaddr *)&sa, sizeof(sa)) < 0) { (struct sockaddr *)&sa, sizeof(sa)) < 0) {
debug("ICMP: failed to relay request to socket"); debug("ICMP: failed to relay request to socket");
} else { } else {
debug("ICMP: echo %s to socket, ID: %i, seq: %i", debug("ICMP: echo request to socket, ID: %i, seq: %i",
(ih->type == ICMP_ECHO) ? "request" : "reply",
id, ntohs(ih->un.echo.sequence)); id, ntohs(ih->un.echo.sequence));
} }
} else if (af == AF_INET6) { } else if (af == AF_INET6) {
@ -222,7 +219,7 @@ int icmp_tap_handler(const struct ctx *c, uint8_t pif, int af,
if (!ih) if (!ih)
return 1; return 1;
if (ih->icmp6_type != 128 && ih->icmp6_type != 129) if (ih->icmp6_type != ICMPV6_ECHO_REQUEST)
return 1; return 1;
iref.id = id = ntohs(ih->icmp6_identifier); iref.id = id = ntohs(ih->icmp6_identifier);
@ -249,8 +246,7 @@ int icmp_tap_handler(const struct ctx *c, uint8_t pif, int af,
(struct sockaddr *)&sa, sizeof(sa)) < 1) { (struct sockaddr *)&sa, sizeof(sa)) < 1) {
debug("ICMPv6: failed to relay request to socket"); debug("ICMPv6: failed to relay request to socket");
} else { } else {
debug("ICMPv6: echo %s to socket, ID: %i, seq: %i", debug("ICMPv6: echo request to socket, ID: %i, seq: %i",
(ih->icmp6_type == 128) ? "request" : "reply",
id, ntohs(ih->icmp6_sequence)); id, ntohs(ih->icmp6_sequence));
} }
} }