c-ares / c-ares / #1303

82%

main: 92%

Build Type

push

travis-ci

Committed by bradh352

Commit Message

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)

Run Details

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

2 existing lines in 2 files now uncovered.

7635 of 9257 relevant lines covered (82.48%)

1979.8 hits per line