aboutgitcodebugslistschat
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2025-12-02 15:02:13 +1100
committerStefano Brivio <sbrivio@redhat.com>2025-12-02 23:07:36 +0100
commitcec1ca854975e9f2201219946bf4aa624abd2ed0 (patch)
tree5a1f1b8a28bb53c0b28e8c338388ce549c0ddcc9
parent653705c10a2133c059b6d90024426220f43aef52 (diff)
downloadpasst-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.c2
-rw-r--r--util.c6
-rw-r--r--util.h4
3 files changed, 6 insertions, 6 deletions
diff --git a/pif.c b/pif.c
index db447b4..3d7a90e 100644
--- a/pif.c
+++ b/pif.c
@@ -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;
diff --git a/util.c b/util.c
index 491e7ff..4beb7c2 100644
--- a/util.c
+++ b/util.c
@@ -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,
diff --git a/util.h b/util.h
index d334f61..7bf0701 100644
--- a/util.h
+++ b/util.h
@@ -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);