summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYang Tse <yangsita@gmail.com>2009-11-23 01:24:17 (GMT)
committerYang Tse <yangsita@gmail.com>2009-11-23 01:24:17 (GMT)
commit36b26039acc147b9b6839b12049707b531b62e32 (patch)
tree03e409cf430ee17291ee4ce01b763dfbf5a49392
parent497afa9acb4d42b4bb186701f0d03a02884ff9b2 (diff)
downloadc-ares-36b26039acc147b9b6839b12049707b531b62e32.zip
c-ares-36b26039acc147b9b6839b12049707b531b62e32.tar.gz
c-ares-36b26039acc147b9b6839b12049707b531b62e32.tar.bz2
- Jakub Hrozek renamed addrttl and addr6ttl structs to ares_addrttl and
ares_addr6ttl in order to prevent name space pollution, along with necessary changes to code base and man pages.This change does not break ABI, there is no need to recompile existing applications. But existing applications using these structs with the old name will need source code adjustments when recompiled using c-ares 1.6.1.
-rw-r--r--CHANGES7
-rw-r--r--RELEASE-NOTES1
-rw-r--r--ares.h13
-rw-r--r--ares_parse_a_reply.36
-rw-r--r--ares_parse_a_reply.c4
-rw-r--r--ares_parse_aaaa_reply.36
-rw-r--r--ares_parse_aaaa_reply.c4
7 files changed, 23 insertions, 18 deletions
diff --git a/CHANGES b/CHANGES
index 6d2a4f1..cfcd2e7 100644
--- a/CHANGES
+++ b/CHANGES
@@ -4,6 +4,13 @@
- Jakub Hrozek fixed more function prototypes in man pages to sync them
with the ones declared in ares.h
+- Jakub Hrozek renamed addrttl and addr6ttl structs to ares_addrttl and
+ ares_addr6ttl in order to prevent name space pollution, along with
+ necessary changes to code base and man pages.This change does not break
+ ABI, there is no need to recompile existing applications. But existing
+ applications using these structs with the old name will need source code
+ adjustments when recompiled using c-ares 1.6.1.
+
* November 20, 2009 (Yang Tse)
- Fixed several function prototypes in man pages that were out of sync
with the ones declared in ares.h
diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 1ecc176..0de52de 100644
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -13,6 +13,7 @@ Changed:
o added ares_parse_srv_reply()
o added ares_parse_txt_reply()
o new --enable-symbol-hiding configure option
+ o addrttl and addr6ttl structs renamed to ares_addrttl and ares_addr6ttl
Fixed:
diff --git a/ares.h b/ares.h
index e79debd..3e45a90 100644
--- a/ares.h
+++ b/ares.h
@@ -416,15 +416,12 @@ struct ares_in6_addr {
} _S6_un;
};
-/*
- * TODO: the structs 'addrttl' and 'addr6ttl' really should get their names
- * prefixed with ares_ to keep them in our own "name space".
- */
-struct addrttl {
+struct ares_addrttl {
struct in_addr ipaddr;
int ttl;
};
-struct addr6ttl {
+
+struct ares_addr6ttl {
struct ares_in6_addr ip6addr;
int ttl;
};
@@ -454,13 +451,13 @@ struct ares_txt_reply {
CARES_EXTERN int ares_parse_a_reply(const unsigned char *abuf,
int alen,
struct hostent **host,
- struct addrttl *addrttls,
+ struct ares_addrttl *addrttls,
int *naddrttls);
CARES_EXTERN int ares_parse_aaaa_reply(const unsigned char *abuf,
int alen,
struct hostent **host,
- struct addr6ttl *addrttls,
+ struct ares_addr6ttl *addrttls,
int *naddrttls);
CARES_EXTERN int ares_parse_ptr_reply(const unsigned char *abuf,
diff --git a/ares_parse_a_reply.3 b/ares_parse_a_reply.3
index 10c0650..f7debb9 100644
--- a/ares_parse_a_reply.3
+++ b/ares_parse_a_reply.3
@@ -23,7 +23,7 @@ ares_parse_a_reply \- Parse a reply to a DNS query of type A
.PP
.B int ares_parse_a_reply(const unsigned char *\fIabuf\fP, int \fIalen\fP,
.B struct hostent **\fIhost\fP,
-.B struct addrttl *\fIaddrttls\fB, int *\fInaddrttls\fB);
+.B struct ares_addrttl *\fIaddrttls\fB, int *\fInaddrttls\fB);
.fi
.SH DESCRIPTION
The
@@ -31,7 +31,7 @@ The
function parses the response to a query of type A into a
.BR "struct hostent"
and/or an array of
-.BR "struct addrttls" .
+.BR "struct ares_addrttls" .
The parameters
.I abuf
and
@@ -51,7 +51,7 @@ and
.IR naddrttls
are both nonnull,
then up to *naddrttls
-.BR "struct addrttl"
+.BR "struct ares_addrttl"
records are stored in the array pointed to by addrttls,
and then *naddrttls is set to the number of records so stored.
Note that the memory for these records is supplied by the caller.
diff --git a/ares_parse_a_reply.c b/ares_parse_a_reply.c
index 3d59c8a..a6ed7dd 100644
--- a/ares_parse_a_reply.c
+++ b/ares_parse_a_reply.c
@@ -54,7 +54,7 @@
int ares_parse_a_reply(const unsigned char *abuf, int alen,
struct hostent **host,
- struct addrttl *addrttls, int *naddrttls)
+ struct ares_addrttl *addrttls, int *naddrttls)
{
unsigned int qdcount, ancount;
int status, i, rr_type, rr_class, rr_len, rr_ttl, naddrs;
@@ -157,7 +157,7 @@ int ares_parse_a_reply(const unsigned char *abuf, int alen,
}
if (naddrs < max_addr_ttls)
{
- struct addrttl * const at = &addrttls[naddrs];
+ struct ares_addrttl * const at = &addrttls[naddrs];
if (aptr + sizeof(struct in_addr) > abuf + alen)
{
status = ARES_EBADRESP;
diff --git a/ares_parse_aaaa_reply.3 b/ares_parse_aaaa_reply.3
index f25cb93..0f77c03 100644
--- a/ares_parse_aaaa_reply.3
+++ b/ares_parse_aaaa_reply.3
@@ -23,7 +23,7 @@ ares_parse_aaaa_reply \- Parse a reply to a DNS query of type AAAA
.PP
.B int ares_parse_aaaa_reply(const unsigned char *\fIabuf\fP, int \fIalen\fP,
.B struct hostent **\fIhost\fP,
-.B struct addr6ttl *\fIaddrttls\fB, int *\fInaddrttls\fB);
+.B struct ares_addr6ttl *\fIaddrttls\fB, int *\fInaddrttls\fB);
.fi
.SH DESCRIPTION
The
@@ -31,7 +31,7 @@ The
function parses the response to a query of type AAAA into a
.BR "struct hostent"
and/or an array of
-.BR "struct addr6ttl" .
+.BR "struct ares_addr6ttl" .
The parameters
.I abuf
and
@@ -51,7 +51,7 @@ and
.IR naddrttls
are both nonnull,
then up to *naddrttls
-.BR "struct addr6ttl"
+.BR "struct ares_addr6ttl"
records are stored in the array pointed to by addrttls,
and then *naddrttls is set to the number of records so stored.
Note that the memory for these records is supplied by the caller.
diff --git a/ares_parse_aaaa_reply.c b/ares_parse_aaaa_reply.c
index 58c6b8a..6d54667 100644
--- a/ares_parse_aaaa_reply.c
+++ b/ares_parse_aaaa_reply.c
@@ -55,7 +55,7 @@
#include "ares_private.h"
int ares_parse_aaaa_reply(const unsigned char *abuf, int alen,
- struct hostent **host, struct addr6ttl *addrttls,
+ struct hostent **host, struct ares_addr6ttl *addrttls,
int *naddrttls)
{
unsigned int qdcount, ancount;
@@ -157,7 +157,7 @@ int ares_parse_aaaa_reply(const unsigned char *abuf, int alen,
}
if (naddrs < max_addr_ttls)
{
- struct addr6ttl * const at = &addrttls[naddrs];
+ struct ares_addr6ttl * const at = &addrttls[naddrs];
if (aptr + sizeof(struct in6_addr) > abuf + alen)
{
status = ARES_EBADRESP;