Salvage tests and fix a leak found in another review: see https://gitlab.winehq.org/wine/wine/-/merge_requests/897#note_9175 points 1 and 2. 3. is why !897 was reverted.
This does not contain the (incorrect) changes to VariantCopyInd, but instead fixes the leak identified in VariantClear (VT_RECORD needs to free pvRecord), and the wrong allocator being used in VariantCopy (it should have been IMalloc, rather than `HeapAlloc`, as shown by the fact IMallocSpy observes these allocations).
This also cleans up some dubious behavior by the test IRecordInfoImpl, which was modifying a VARIANT that it did not own or receive as an argument in the middle of VariantClear. This was likely undefined behavior, and in any case concealed the heap leak.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1035
If the canonical hostname was "localhost", the address info was not
freed.
Don't bother checking whether ai_canonname is NULL because if there is
no canonical name, getaddrinfo must set ai_canonname to the input name.
Fixes: ca5a6d07dc92ba631b178ec175e6b3fd5295e3d6
--
v2: wineboot: Fix a memory leak in create_computer_name_keys.
https://gitlab.winehq.org/wine/wine/-/merge_requests/8918
If the canonical hostname was "localhost", the address info was not
freed.
Also, since we're touching the same line that defines the address info
hints, make the hints `static const` so that the compiler knows it's
safe to put them in read-only memory if it wants.
Fixes: ca5a6d07dc92ba631b178ec175e6b3fd5295e3d6
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/8918