use undeprecated functions in c-ares

[tchaikov]

c-ares v1.28.1 deprecated a bunch of functions. for instance, ares_gethostbyname() is deprecated in favor of ares_getaddrinfo()

/home/kefu/dev/scylladb/seastar/src/net/dns.cc:265:9: warning: 'ares_gethostbyname' is deprecated: Use ares_getaddrinfo instead [-Wdeprecated-declarations]
  265 |         ares_gethostbyname(_channel, p->name.c_str(), af, [](void* arg, int status, int timeouts, ::hostent* host) {
      |         ^
/usr/include/ares.h:592:14: note: 'ares_gethostbyname' has been explicitly marked deprecated here
  592 | CARES_EXTERN CARES_DEPRECATED_FOR(ares_getaddrinfo) void ares_gethostbyname(
      |              ^
/usr/include/ares.h:145:22: note: expanded from macro 'CARES_DEPRECATED_FOR'
  145 |       __attribute__((deprecated("Use " #f " instead")))
      |                      ^
/home/kefu/dev/scylladb/seastar/src/net/dns.cc:357:22: warning: 'ares_parse_srv_reply' is deprecated: Use ares_dns_parse instead [-Wdeprecated-declarations]
  357 |             status = ares_parse_srv_reply(buf, len, &start);
      |                      ^

[tchaikov]

see also c-ares/c-ares@5fd3fc3

[dawmd]

It's worth noting that among the deprecated functions there is ares_fds, which may crash the system (cf. https://c-ares.org/ares_fds.html#NOTES):

The select(2) call which takes the fd_set parameter has significant limitations which can impact modern systems. The limitations can vary from system to system, but in general if the file descriptor value itself is greater than 1024 (not the count but the actual value), this can lead to ares_fds writing out of bounds which will cause a system crash. In modern networking clients, it is not unusual to have file descriptor values above 1024, especially when a library is pulled in as a dependency into a larger project.

c-ares does not attempt to detect this condition to prevent crashes due to both implementation-defined behavior in the OS as well as integrator-controllable tunables which may impact the limits.