diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2024-08-21 14:19:57 +1000 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2024-08-21 11:59:29 +0200 |
commit | e6feb5a892b4efc2b149883f58915154d6eb349f (patch) | |
tree | a7f912dad260d2a5c14ffdbf7aaed62283d6e3de /icmp.c | |
parent | 32c386834dd6a30be60f8b90a8d5a17988c91997 (diff) | |
download | passt-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.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -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); |