allow freeaddrinfo of arbitrary sublists of addrinfo list
authorRich Felker <dalias@aerifal.cx>
Fri, 5 Oct 2018 00:27:17 +0000 (20:27 -0400)
committerRich Felker <dalias@aerifal.cx>
Fri, 5 Oct 2018 00:27:17 +0000 (20:27 -0400)
commitd1395c43c019aec6b855cf3c656bf47c8a719e7f
treec7c070b4887d17832631c2868ea6f9a757194dab
parent7bf773a8f9b97875ba67b646c1681ac5ca22016f
allow freeaddrinfo of arbitrary sublists of addrinfo list

the specification for freeaddrinfo allows it to be used to free
"arbitrary sublists" of the list returned by getaddrinfo. it's not
clearly stated how such sublists come into existence, but the
interpretation seems to be that the application can edit the ai_next
pointers to cut off a portion of the list and then free it.

actual freeing of individual list slots is contrary to the design of
our getaddrinfo implementation, which has no failure paths after
making a single allocation, so that light callers can avoid linking
realloc/free. freeing individual slots is also incompatible with
sharing the string for ai_canonname, which the current implementation
does despite no requirement that it be present except on the first
result. so, rather than actually freeing individual slots, provide a
way to find the start of the allocated array, and reference-count it,
freeing the memory all at once after the last slot has been freed.

since the language in the spec is "arbitrary sublists", no provision
for handling other constructs like multiple lists glued together,
circular links, etc. is made. presumably passing such a construct to
freeaddrinfo produces undefined behavior.
src/network/freeaddrinfo.c
src/network/getaddrinfo.c
src/network/lookup.h