oops 5 bytes makes 4 letters plus zero byte

fix warning

the _num_chars() function is not used, removing

to prevent compier warnings, we only declare len if we have code
that uses it

Gisle: free used memory better

Gisle cleaned up remaining host resolve re-org issues

include header for our printfs

reverted bad win32 fix

fixed the MSVC build.

length limit the sscanf() parsing to prevent buffer overflow

ah, simplified my latest change more

fix for systems without gethostbyname_r()

Replaced all uses of sprintf() with the safer snprintf(). It is just a
precaution to prevent mistakes to lead to buffer overflows.

made the Curl_he2ai() take the port number as an int intead, to avoid lots
of typecasts all over

use snprintf() to be on the safe side

typecasts to prevent warnings

only use sockaddr_in6 on ipv6-enabled hosts

socks proxy support even when libcurl is built ipv6-enabled

typecast to prevent warning

use Curl_addrinfo, not 'struct addrinfo'

fixed problems I missed to fix from my cleanup

prevent warning

Yet another resolve code re-org

Source cleanups. The major one being that we now _always_ use a Curl_addrinfo
linked list for name resolved data, even on hosts/systems with only IPv4
stacks as this simplifies a lot of code.

static functions are better not Curl_ prefixed to make their static
status more obvious