c-ares / c-ares / #1299

80%

main: 92%

push

travis-ci

mem leak due to `ares__hosts_entry_to_hostent()` allocation strategy (#824)

`ares__hosts_entry_to_hostent()` would allocate a separate buffer for
each address, but `ares_free_hostent()` expects a single allocation to
hold all addresses.

This PR fixes this issue and simplifies the logic by using the
already-existing `ares__addrinfo2hostent()` to write the hostent instead
of coming up with yet another way to write the structure.

Fixes #823
Fix By: Brad House (@bradh352)

15 of 22 new or added lines in 1 file covered. (68.18%)

8381 of 10458 relevant lines covered (80.14%)

3504.75 hits per line