diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2025-12-02 15:02:13 +1100 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2025-12-02 23:07:36 +0100 |
| commit | cec1ca854975e9f2201219946bf4aa624abd2ed0 (patch) | |
| tree | 5a1f1b8a28bb53c0b28e8c338388ce549c0ddcc9 | |
| parent | 653705c10a2133c059b6d90024426220f43aef52 (diff) | |
| download | passt-cec1ca854975e9f2201219946bf4aa624abd2ed0.tar passt-cec1ca854975e9f2201219946bf4aa624abd2ed0.tar.gz passt-cec1ca854975e9f2201219946bf4aa624abd2ed0.tar.bz2 passt-cec1ca854975e9f2201219946bf4aa624abd2ed0.tar.lz passt-cec1ca854975e9f2201219946bf4aa624abd2ed0.tar.xz passt-cec1ca854975e9f2201219946bf4aa624abd2ed0.tar.zst passt-cec1ca854975e9f2201219946bf4aa624abd2ed0.zip | |
util: Rename sock_l4_dualstack() to sock_l4_dualstack_any()
Stefano correctly noted that the fact a socket is dual-stack doesn't
necessarily imply that it is bound to a wildcard address. While that's the
only case we use for dual-stack sockets, there may be others. Therefore
rename this function to make it clearer that it always uses a wildcard
bind.
Suggested-by: Stefano Brivio <sbrivio@redhat.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
| -rw-r--r-- | pif.c | 2 | ||||
| -rw-r--r-- | util.c | 6 | ||||
| -rw-r--r-- | util.h | 4 |
3 files changed, 6 insertions, 6 deletions
@@ -82,7 +82,7 @@ int pif_sock_l4(const struct ctx *c, enum epoll_type type, uint8_t pif, ASSERT(pif_is_socket(pif)); if (!addr) { - ref.fd = sock_l4_dualstack(c, type, port, ifname); + ref.fd = sock_l4_dualstack_any(c, type, port, ifname); } else { union sockaddr_inany sa; @@ -198,7 +198,7 @@ int sock_l4(const struct ctx *c, enum epoll_type type, } /** - * sock_l4_dualstack() - Create a dual stack socket bound with wildcard address + * sock_l4_dualstack_any() - Create dualstack socket bound to :: and 0.0.0.0 * @c: Execution context * @type: epoll type * @port Port to bind to (:: and 0.0.0.0) @@ -208,8 +208,8 @@ int sock_l4(const struct ctx *c, enum epoll_type type, * * A dual stack socket is effectively bound to both :: and 0.0.0.0. */ -int sock_l4_dualstack(const struct ctx *c, enum epoll_type type, - in_port_t port, const char *ifname) +int sock_l4_dualstack_any(const struct ctx *c, enum epoll_type type, + in_port_t port, const char *ifname) { union sockaddr_inany sa = { .sa6.sin6_family = AF_INET6, @@ -210,8 +210,8 @@ union sockaddr_inany; int sock_l4(const struct ctx *c, enum epoll_type type, const union sockaddr_inany *sa, const char *ifname); -int sock_l4_dualstack(const struct ctx *c, enum epoll_type type, - in_port_t port, const char *ifname); +int sock_l4_dualstack_any(const struct ctx *c, enum epoll_type type, + in_port_t port, const char *ifname); int sock_unix(char *sock_path); void sock_probe_features(struct ctx *c); long timespec_diff_ms(const struct timespec *a, const struct timespec *b); |
