aboutgitcodebugslistschat
path: root/icmp.c
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2024-08-21 14:19:57 +1000
committerStefano Brivio <sbrivio@redhat.com>2024-08-21 11:59:29 +0200
commite6feb5a892b4efc2b149883f58915154d6eb349f (patch)
treea7f912dad260d2a5c14ffdbf7aaed62283d6e3de /icmp.c
parent32c386834dd6a30be60f8b90a8d5a17988c91997 (diff)
downloadpasst-e6feb5a892b4efc2b149883f58915154d6eb349f.tar
passt-e6feb5a892b4efc2b149883f58915154d6eb349f.tar.gz
passt-e6feb5a892b4efc2b149883f58915154d6eb349f.tar.bz2
passt-e6feb5a892b4efc2b149883f58915154d6eb349f.tar.lz
passt-e6feb5a892b4efc2b149883f58915154d6eb349f.tar.xz
passt-e6feb5a892b4efc2b149883f58915154d6eb349f.tar.zst
passt-e6feb5a892b4efc2b149883f58915154d6eb349f.zip
treewide: Use "our address" instead of "forwarding address"
The term "forwarding address" to indicate the local-to-passt address was well-intentioned, but ends up being kinda confusing. As discussed on a recent call, let's try "our" instead. (While we're there correct an error in flow_initiate_af()s comments where we referred to parameters by the wrong name). Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'icmp.c')
-rw-r--r--icmp.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/icmp.c b/icmp.c
index cb81c76..f514dbc 100644
--- a/icmp.c
+++ b/icmp.c
@@ -125,13 +125,13 @@ void icmp_sock_handler(const struct ctx *c, union epoll_ref ref)
ini->eport, seq);
if (pingf->f.type == FLOW_PING4) {
- const struct in_addr *saddr = inany_v4(&ini->faddr);
+ const struct in_addr *saddr = inany_v4(&ini->oaddr);
const struct in_addr *daddr = inany_v4(&ini->eaddr);
ASSERT(saddr && daddr); /* Must have IPv4 addresses */
tap_icmp4_send(c, *saddr, *daddr, buf, n);
} else if (pingf->f.type == FLOW_PING6) {
- const struct in6_addr *saddr = &ini->faddr.a6;
+ const struct in6_addr *saddr = &ini->oaddr.a6;
const struct in6_addr *daddr = &ini->eaddr.a6;
tap_icmp6_send(c, saddr, daddr, buf, n);