Matt Caswell

Abandoning this PR

Added "tests:deferred" label because we really need better testing of this - but can't really do it until we have some mechanism for a test server within the test harness.

Lifting the hold on this since the controversial stuff has been moved elsewhere.

@t8m please reconfirm

> Header: > Version = TLS 1.0 (0x301) This is the record layer version and is ignored by all correct implementations. It has no bearing on the offered TLS versions...

> I specified tlsv1.2 and it used 1.3 in curl. You should see the details capturing the traffic with wireshark. This seems to be quite a different issue to the...

Really happy to see that we now have tests for this. I'm also ok for the tests to be in a separate PR - it will probably make things easier...

Closing since this is marked as resolved.

if (!sk_IPAddressFamily_push(addr, f1)) goto end; This pushes `f1` into the `addr` stack. IPAddressFamily_free(f1); sk_IPAddressFamily_pop_free(addr, IPAddressFamily_free); The first line frees `f1`. The second line calls `IPAddressFamily_free` for all elements in the...

I'd suggest running it in a debugger to figure out what's going wrong.