Some systems may be lacking getservbyport; the previous ifdef wasn't
quite enough since it still assumed that struct servent was defined.
Simply remove the possibility to return non-numeric services in
getnameinfo; no caller of getnameinfo within libavformat
currently try to use getnameinfo for retrieving the port number without
NI_NUMERICSERV.
The issue that the code currently relied on struct servent being defined
was pointed out by Clément Gregoire.
---
configure | 2 --
libavformat/os_support.c | 11 ++---------
2 files changed, 2 insertions(+), 11 deletions(-)
diff --git a/configure b/configure
index 8d44768..3dd28c1 100755
--- a/configure
+++ b/configure
@@ -1513,7 +1513,6 @@ SYSTEM_FUNCS="
GetProcessMemoryInfo
GetProcessTimes
getrusage
- getservbyport
GetSystemTimeAsFileTime
gettimeofday
gmtime_r
@@ -4240,7 +4239,6 @@ fi
if ! disabled network; then
check_func getaddrinfo $network_extralibs
- check_func getservbyport $network_extralibs
check_func inet_aton $network_extralibs
check_type netdb.h "struct addrinfo"
diff --git a/libavformat/os_support.c b/libavformat/os_support.c
index 650baea..e9b1f88 100644
--- a/libavformat/os_support.c
+++ b/libavformat/os_support.c
@@ -204,16 +204,9 @@ int ff_getnameinfo(const struct sockaddr *sa, int salen,
}
if (serv && servlen > 0) {
- struct servent *ent = NULL;
-#if HAVE_GETSERVBYPORT
if (!(flags & NI_NUMERICSERV))
- ent = getservbyport(sin->sin_port, flags & NI_DGRAM ? "udp" :
"tcp");
-#endif /* HAVE_GETSERVBYPORT */
-
- if (ent)
- snprintf(serv, servlen, "%s", ent->s_name);
- else
- snprintf(serv, servlen, "%d", ntohs(sin->sin_port));
+ return EAI_FAIL;
+ snprintf(serv, servlen, "%d", ntohs(sin->sin_port));
}
return 0;
--
2.4.9 (Apple Git-60)
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
