From 5cefef591ef6c0fd1c514202b0fcfbe466cab873 Mon Sep 17 00:00:00 2001 From: Alistair Leslie-Hughes Date: Thu, 26 Jan 2023 08:37:20 +1100 Subject: rtp-recv: Use getaddrinfo to improve support for ipv6. Part-of: --- src/modules/rtp/module-rtp-recv.c | 38 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 38 insertions(+) diff --git a/src/modules/rtp/module-rtp-recv.c b/src/modules/rtp/module-rtp-recv.c index a67a993d1..9babf00cb 100644 --- a/src/modules/rtp/module-rtp-recv.c +++ b/src/modules/rtp/module-rtp-recv.c @@ -29,6 +29,9 @@ #include #include #include +#ifdef HAVE_NETDB_H +#include +#endif #include #include @@ -677,9 +680,13 @@ static void check_death_event_cb(pa_mainloop_api *m, pa_time_event *t, const str int pa__init(pa_module*m) { struct userdata *u; pa_modargs *ma = NULL; +#if defined(HAVE_GETADDRINFO) + struct addrinfo *sap_addrinfo = NULL; +#else struct sockaddr_in sa4; #ifdef HAVE_IPV6 struct sockaddr_in6 sa6; +#endif #endif struct sockaddr *sa; socklen_t salen; @@ -696,6 +703,27 @@ int pa__init(pa_module*m) { sap_address = pa_modargs_get_value(ma, "sap_address", DEFAULT_SAP_ADDRESS); +#if defined(HAVE_GETADDRINFO) + { + struct addrinfo hints; + char *service; + + pa_zero(hints); + + service = pa_sprintf_malloc("%d", htons(SAP_PORT)); + + hints.ai_flags = AI_NUMERICHOST; + if (getaddrinfo(sap_address, service, &hints, &sap_addrinfo) != 0) { + pa_xfree(service); + pa_log("Invalid SAP address '%s'", sap_address); + goto fail; + } + pa_xfree(service); + + sa = sap_addrinfo->ai_addr; + salen = sap_addrinfo->ai_addrlen; + } +#else if (inet_pton(AF_INET, sap_address, &sa4.sin_addr) > 0) { sa4.sin_family = AF_INET; sa4.sin_port = htons(SAP_PORT); @@ -712,6 +740,7 @@ int pa__init(pa_module*m) { pa_log("Invalid SAP address '%s'", sap_address); goto fail; } +#endif latency_msec = DEFAULT_LATENCY_MSEC; if (pa_modargs_get_value_u32(ma, "latency_msec", &latency_msec) < 0 || latency_msec < 1 || latency_msec > 300000) { @@ -739,9 +768,18 @@ int pa__init(pa_module*m) { pa_modargs_free(ma); +#if defined(HAVE_GETADDRINFO) + freeaddrinfo(sap_addrinfo); +#endif + return 0; fail: +#if defined(HAVE_GETADDRINFO) + if (sap_addrinfo) + freeaddrinfo(sap_addrinfo); +#endif + if (ma) pa_modargs_free(ma); -- cgit v1.2.1