aboutgitcodebugslistschat
diff options
context:
space:
mode:
authorStefano Brivio <sbrivio@redhat.com>2022-10-26 17:45:48 +0200
committerStefano Brivio <sbrivio@redhat.com>2022-10-27 00:18:18 +0200
commitb062ee47d1d587ad15106d09aec4d1710f256cd6 (patch)
tree2c51d1c3a2e119f28df3c06d6514b67c0f43bf33
parent947d756747bfe938d95c3cbc163d148b54e03d68 (diff)
downloadpasst-b062ee47d1d587ad15106d09aec4d1710f256cd6.tar
passt-b062ee47d1d587ad15106d09aec4d1710f256cd6.tar.gz
passt-b062ee47d1d587ad15106d09aec4d1710f256cd6.tar.bz2
passt-b062ee47d1d587ad15106d09aec4d1710f256cd6.tar.lz
passt-b062ee47d1d587ad15106d09aec4d1710f256cd6.tar.xz
passt-b062ee47d1d587ad15106d09aec4d1710f256cd6.tar.zst
passt-b062ee47d1d587ad15106d09aec4d1710f256cd6.zip
icmp: Add debugging messages for handled replies and requests
...instead of just reporting errors. Signed-off-by: Stefano Brivio <sbrivio@redhat.com> Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
-rw-r--r--icmp.c30
1 files changed, 25 insertions, 5 deletions
diff --git a/icmp.c b/icmp.c
index 233acf9..9caa7e6 100644
--- a/icmp.c
+++ b/icmp.c
@@ -88,6 +88,7 @@ void icmp_sock_handler(const struct ctx *c, union epoll_ref ref,
struct icmp6hdr *ih = (struct icmp6hdr *)buf;
id = ntohs(ih->icmp6_identifier);
+ seq = ntohs(ih->icmp6_sequence);
/* If bind() fails e.g. because of a broken SELinux policy, this
* might happen. Fix up the identifier to match the sent one.
@@ -97,14 +98,15 @@ void icmp_sock_handler(const struct ctx *c, union epoll_ref ref,
/* In PASTA mode, we'll get any reply we send, discard them. */
if (c->mode == MODE_PASTA) {
- seq = ntohs(ih->icmp6_sequence);
-
if (icmp_id_map[V6][id].seq == seq)
return;
icmp_id_map[V6][id].seq = seq;
}
+ debug("ICMPv6: echo %s to tap, ID: %i, seq: %i",
+ (ih->icmp6_type == 128) ? "request" : "reply", id, seq);
+
tap_icmp6_send(c, &sr6->sin6_addr,
tap_ip6_daddr(c, &sr6->sin6_addr), buf, n);
} else {
@@ -112,11 +114,12 @@ void icmp_sock_handler(const struct ctx *c, union epoll_ref ref,
struct icmphdr *ih = (struct icmphdr *)buf;
id = ntohs(ih->un.echo.id);
+ seq = ntohs(ih->un.echo.sequence);
+
if (id != iref->icmp.id)
ih->un.echo.id = htons(iref->icmp.id);
if (c->mode == MODE_PASTA) {
- seq = ntohs(ih->un.echo.sequence);
if (icmp_id_map[V4][id].seq == seq)
return;
@@ -124,6 +127,9 @@ void icmp_sock_handler(const struct ctx *c, union epoll_ref ref,
icmp_id_map[V4][id].seq = seq;
}
+ debug("ICMP: echo %s to tap, ID: %i, seq: %i",
+ (ih->type == ICMP_ECHO) ? "request" : "reply", id, seq);
+
tap_icmp4_send(c, sr4->sin_addr.s_addr, tap_ip4_daddr(c),
buf, n);
}
@@ -175,14 +181,21 @@ int icmp_tap_handler(const struct ctx *c, int af, const void *addr,
}
icmp_id_map[V4][id].sock = s;
+
+ debug("ICMP: new socket %i for echo ID %i", s, id);
}
icmp_id_map[V4][id].ts = now->tv_sec;
bitmap_set(icmp_act[V4], id);
sa.sin_addr = *(struct in_addr *)addr;
if (sendto(s, ih, sizeof(*ih) + plen, MSG_NOSIGNAL,
- (struct sockaddr *)&sa, sizeof(sa)) < 0)
+ (struct sockaddr *)&sa, sizeof(sa)) < 0) {
debug("ICMP: failed to relay request to socket");
+ } else {
+ debug("ICMP: echo %s to socket, ID: %i, seq: %i",
+ (ih->type == ICMP_ECHO) ? "request" : "reply",
+ id, ntohs(ih->un.echo.sequence));
+ }
} else if (af == AF_INET6) {
union icmp_epoll_ref iref = { .icmp.v6 = 1 };
struct sockaddr_in6 sa = {
@@ -214,14 +227,21 @@ int icmp_tap_handler(const struct ctx *c, int af, const void *addr,
}
icmp_id_map[V6][id].sock = s;
+
+ debug("ICMPv6: new socket %i for echo ID %i", s, id);
}
icmp_id_map[V6][id].ts = now->tv_sec;
bitmap_set(icmp_act[V6], id);
sa.sin6_addr = *(struct in6_addr *)addr;
if (sendto(s, ih, sizeof(*ih) + plen, MSG_NOSIGNAL,
- (struct sockaddr *)&sa, sizeof(sa)) < 1)
+ (struct sockaddr *)&sa, sizeof(sa)) < 1) {
debug("ICMPv6: failed to relay request to socket");
+ } else {
+ debug("ICMPv6: echo %s to socket, ID: %i, seq: %i",
+ (ih->icmp6_type == 128) ? "request" : "reply",
+ id, ntohs(ih->icmp6_sequence));
+ }
}
return 1;