ares_inet_ntop: reapply changes from previous c-ares version (III)

- Replace 'u_char' with 'unsigned char'.
- Replace 'u_int' with 'unsigned int'.
- use macros ERRNO and SET_ERRNO() for errno handling.

ares_inet_ntop: reapply changes from previous c-ares version (II)

- Remove rcsid.
- Adjust header file inclusions.
- ares_inet_ntop used only on systems without a proper inet_ntop function.

ares_inet_ntop: reapply changes from previous c-ares version (I)

- Replace tabs with spaces.
- Use ANSI C style for function declarations and definitions.
- Use sizeof with parentheses.

ares_inet_ntop: fix off by one error triggering out of bounds write

ares_inet_ntop would trigger an out of bounds write when the representation
of the address required 15 characters, due to not taking in account null
termination character.

Full import of inet_ntop.c from bind-9.5.3rc1 to pull additional fixes.

ares_nowarn: add conditional inclusion of assert.h header

fix compiler warning: conversion may lose significant bits

ares_inet_net_pton: fix non-rejection of some malformed literals

ares_inet_net_pton would return wrong values when excessively large,
and invalid, netmasks are used. Fixes are from bind-9.5.3rc1,
issue also described in the WLB-2008080064 advisory.

setup_once: provide ISASCII macro

configure: inet_net_pton function check adjustments

Define HAVE_INET_NET_PTON only when system's inet_net_pton function is IPv6
capable and is not affected by the WLB-2008080064 advisory.

HAVE_INET_NET_PTON_IPV6 is no longer defined nor used.

ares_init: fix detection of semicolon comments in resolv.conf

File resolv.conf may either use a hash '#' or a semicolon ';' character as an
indication that the rest of the line is a comment.  This fixes not recognizing
the semicolon as a valid comment indicator in resolv.conf.

version: start working on 1.7.5

release-preps: CHANGES and RELEASE-NOTES synced

ares_set_local_*: added in 1.7.4, not before

build: provide SIZEOF_SIZE_T definition for non-configure builds

build: config.dos renamed to config-dos.h

build: provide SIZEOF_SIZE_T netware definition

ares_gethostbyaddr: fix compiler warning: conversion may lose significant bits

configure: undo using autobuilds to temporarily verify strict aliasing warnings.

fix compiler warning: rounding, sign extension, or loss of accuracy may result

ares_parse_a_reply: fix CNAME response parsing

Reply to a CNAME query doesn't contain addresses, causing
ares_parse_a_reply() to bail out with ARES_ENODATA

Bug: http://groups.google.com/group/nodejs/browse_thread/thread/a1268c9ea5e9ad9b

fix compiler warning: conversion may lose significant bits

atoi: remove atoi usage

ares_init: fix compiler warning: conversion may lose significant bits

configure: fix autoconf warning

inet_pton: fix compiler warning

configure: use autobuilds to temporarily verify strict aliasing warnings.

Temporarily, When cross-compiling with gcc 3.0 or later, enable strict aliasing
rules and warnings. Given that cross-compiled targets autobuilds do not run the
test-suite, there is no risk of running code that violates strict aliasing rules

ares_getnameinfo: Partially revert commit 85520d66e0ac7ac73411bc25e98769a88b2f

Upon socket address family and length validation failure return ARES_ENOTIMP
in callback again, this is the error code documented in man page and used
mostly all over the library.

ares_getnameinfo: Validate socket address family and length.

Validate socket address family and that the socket address length is appropriate
for the specified family. Failure is reported with ARES_EBADFAMILY in callback.

ares_getnameinfo: fix two compiler warnings

Added another VS10 version string

Fix GCC 4 compiler warning 'dereferencing type-punned pointer might break strict-aliasing rules'.

Revert commit 494274e653936335c255a47599970de3df21e7c4

configure: fix autoconf 2.68 warning: no AC_LANG_SOURCE call detected in body

Fix compiler warning: array subscript has type 'char'

Fix GCC 4 compiler warning 'dereferencing type-punned pointer might break strict-aliasing rules'.

Revert following commits:
  07bc7ea79509bcc9ef6e09151e81766ed00d3392
  3392a50ea3f8573ea4b7a9d82b9833dab60cb0e9
  9912637d32c9987719a1ea12db591aee2941891c

The purpose of the whole patch was to silence a compiler warning triggered
with GCC 4 on file ares_process.c  The specific compiler warning was
'dereferencing type-punned pointer might break strict-aliasing rules'.

A simpler patch will follow to equally silence the warning.

ares_options: reorder header inclusions to make inclusion of
ares_private.h the last included one again.

adig: fix NAPTR parsing

I ran across a small "issue" in your adig example.

It is simply the last part of the NAPTR record, the replacement element,
which is not a string, as currently handled in adig, but a domain name.

ares_save_options: assignments instead of memcpy

init_by_options: don't copy an empty sortlist

If there aren't any sort items to copy, don't bother. Without this
little precaution it would do a malloc(0) which causes undefined
behaviors and is frowned upon by curl's memdebug-system.

Minor Watcom makefile tweaks.

Fix lookup with HOSTALIASES set.

ares__read_line returns ARES_EOF when it reaches the end of the
file. This will happen every time when reading to the end of the
HOSTALIASES file. Unfortunately single_domain treats this error as
being fatal.

Signed-off-by: Mike Crowe <mac@mcrowe.com>

Add missing break that caused get_ares_servers to fail.

Reported-by: Ning Dong <flintning@163.com>
Signed-off-by: Ben Greear <greearb@candelatech.com>

configure: werror related adjustments

Added copyright string to ares_version.h and make use of it in other files.

Block created ares_build.h for NetWare to avoid usage from other platforms.

Fix to overwrite default libname.

Some more Watcom makefile massage ...

Some more Watcom makefile massage ...

sock-addr-storage:  Detect and deal with lack of .ss_family member.

AIX, at least, does not have sockaddr_storage.ss_family member.
Detect this in the configure logic and use proper #ifdefs in the
ares_process logic.

Signed-off-by: Ben Greear <greearb@candelatech.com>
Tested-by: Tor Arntsen <tor@spacetec.no>

Added Watcom makefile based on libcurl's Makefile.Watcom.

typo:  Fix compile bug for platforms that don't have sockaddr_storage.

Bug was introduced by me in previous commit.

Signed-off-by: Ben Greear <greearb@candelatech.com>

Fix aliasing warning in gcc 4.4.4 (at least).

Should be no functional change, though the code gets a bit
ugglier.

Signed-off-by: Ben Greear <greearb@candelatech.com>

ares_set_servers_csv: use ISDIGIT

The IS*() set of macros are preferred to the regular is*() functions as
they help us avoid the most common pitfalls.

cast arg to isdigit to int

Looks like it might silence a warning on Netware build.

Signed-off-by: Ben Greear <greearb@candelatech.com>

remove all uses of uint32_t

Previous fix forgot a few.

Signed-off-by: Ben Greear <greearb@candelatech.com>

fix signed v/s unsigned casts warning in ares_gethostbyaddr.c

Signed-off-by: Ben Greear <greearb@candelatech.com>

local-bind-fixup:  Fix inet_pton warning.

Conditionally include <arpa/inet.h> for inet_pton
headers.

Signed-off-by: Ben Greear <greearb@candelatech.com>

build:  Enable compiling with -Werror.

This helps find compile warnings because they simply break
the build.

To use:
./configure --enable-warnings --enable-werror

Signed-off-by: Ben Greear <greearb@candelatech.com>

ipv6: Fix some build issues related to the local-bind feature.

Signed-off-by: Ben Greear <greearb@candelatech.com>

Replaced uint32_t with unsigned int to fix broken builds on a couple of platforms.

local-bind:  Support binding to local interface/IPs

Add 3 new functions to set the local binding for the out-going
socket connection, and add ares_set_servers_csv() to set a
list of servers at once as a comma-separated string.

Signed-off-by: Ben Greear <greearb@candelatech.com>

version: now start on 1.7.4

fix memory leak in ares_getnameinfo

changelogs: updated for 1.7.3

init: allow c-ares to work on Android OS

changelog: fill in the 1.7.2 changes

added another pdf to ignore

add ares_parse_mx_reply.c to VS dsp file

tarball: add $(CSOURCES) $(HHEADERS) to EXTRA_DIST

It's not clear to me why we need this, but we apparently may
otherwise not get all files bundled in the dist tarball.

version: start working on 1.7.3

RELEASE-NOTES: 1.7.2 details added

ares_init: Last, not first instance of domain or search should win

style: make code less than 80 columns wide

improve alternative definition of bool to use enum instead of unsigned char

fix VS2010 compiler warnings

added ares_parse_mx_reply

repair the file mode

remove all $Id$ lines

remove all .cvsignore files

spell fix

reported by Gregor Jasny on the mailing list

Fix a couple of typos and grammar nits.

ignore the GPG signature files too

start the journey towards 1.7.2

no longer CVS tagging

ignore generated PDFs

1.7.1

made README the primary readme file

... and did README.cares to contain a historic reason etc.

s/CVS/git

git now, not CVS

ignore lots of generated files

Fix warnings for clang

replaced intel compiler option -no-ansi-alias with -fno-strict-aliasing

update outdated serial number

fix compiler warning

watt32 compilation fix

Added another VS10 version string

fix line break

removed usage of 's6_addr', fixing compilation issue triggered with no
longer using 'in6_addr' but only our 'ares_in6_addr' struct

Daniel Johnson provided fixes for building with the clang compiler