Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hostip: move code to resolve IP address literals to Curl_resolv #4798

Closed
wants to merge 1 commit into from

Conversation

MarcelRaad
Copy link
Member

The code was duplicated in the various resolver backends.

Also, it was called after the call to Curl_ipvalid, which matters in
case of CURLRES_IPV4 when called from connect.c:bindlocal. This
caused test 1048 to fail on classic MinGW.

The code ignores conn->ip_version as done previously in the
individual resolver backends.

Move the call to the resolver_start callback up to appease test 655,
which wants it to be called also for literal addresses.

The code was duplicated in the various resolver backends.

Also, it was called after the call to `Curl_ipvalid`, which matters in
case of `CURLRES_IPV4` when called from `connect.c:bindlocal`. This
caused test 1048 to fail on classic MinGW.

The code ignores `conn->ip_version` as done previously in the
individual resolver backends.

Move the call to the `resolver_start` callback up to appease test 655,
which wants it to be called also for literal addresses.

Closes
@MarcelRaad MarcelRaad added name lookup DNS and related tech tests tidy-up labels Jan 8, 2020
Copy link
Member

@jay jay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but note I reviewed it with whitespace ignored ?w=1

@MarcelRaad MarcelRaad deleted the hostip_literals branch January 10, 2020 17:17
@lock lock bot locked as resolved and limited conversation to collaborators Apr 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

3 participants