* Get rid of some type-punning warnings
@ 2002-12-28 7:43 Andreas Jaeger
2002-12-28 13:06 ` Ulrich Drepper
0 siblings, 1 reply; 3+ messages in thread
From: Andreas Jaeger @ 2002-12-28 7:43 UTC (permalink / raw)
To: GNU libc hacker
Ok to commit the appended patch? It gets rid of these warnings:
nss_dns/dns-host.c:176: warning: dereferencing type-punned pointer will break strict-aliasing rules
nss_dns/dns-host.c:193: warning: dereferencing type-punned pointer will break strict-aliasing rules
nss_dns/dns-host.c:311: warning: dereferencing type-punned pointer will break strict-aliasing rules
nss_dns/dns-host.c:317: warning: dereferencing type-punned pointer will break strict-aliasing rules
nss_dns/dns-network.c:126: warning: dereferencing type-punned pointer will break strict-aliasing rules
nss_dns/dns-network.c:196: warning: dereferencing type-punned pointer will break strict-aliasing rules
Andreas
2002-12-28 Andreas Jaeger <aj@suse.de>
* resolv/nss_dns/dns-network.c (_nss_dns_getnetbyaddr_r): Likewise.
(_nss_dns_getnetbyname_r): Likewise.
* resolv/nss_dns/dns-host.c (_nss_dns_gethostbyaddr_r): Use union
type to avoid strict aliasing problem.
(_nss_dns_gethostbyname2_r): Likewise.
============================================================
Index: resolv/nss_dns/dns-host.c
--- resolv/nss_dns/dns-host.c 17 Oct 2002 21:48:48 -0000 1.36
+++ resolv/nss_dns/dns-host.c 28 Dec 2002 15:35:11 -0000
@@ -132,7 +132,12 @@ _nss_dns_gethostbyname2_r (const char *n
char *buffer, size_t buflen, int *errnop,
int *h_errnop)
{
- querybuf *host_buffer, *orig_host_buffer;
+ union
+ {
+ querybuf *buf;
+ void *ptr;
+ } host_buffer;
+ querybuf *orig_host_buffer;
char tmp[NS_MAXDNAME];
int size, type, n;
const char *cp;
@@ -170,10 +175,10 @@ _nss_dns_gethostbyname2_r (const char *n
&& (cp = res_hostalias (&_res, name, tmp, sizeof (tmp))) != NULL)
name = cp;
- host_buffer = orig_host_buffer = (querybuf *) alloca (1024);
+ host_buffer.buf = orig_host_buffer = (querybuf *) alloca (1024);
- n = __libc_res_nsearch (&_res, name, C_IN, type, host_buffer->buf,
- 1024, (u_char **) &host_buffer);
+ n = __libc_res_nsearch (&_res, name, C_IN, type, host_buffer.buf->buf,
+ 1024, (u_char **) &host_buffer.ptr);
if (n < 0)
{
enum nss_status status = (errno == ECONNREFUSED
@@ -188,14 +193,14 @@ _nss_dns_gethostbyname2_r (const char *n
by having the RES_USE_INET6 bit in _res.options set, we try
another lookup. */
if (af == AF_INET6 && (_res.options & RES_USE_INET6))
- n = __libc_res_nsearch (&_res, name, C_IN, T_A, host_buffer->buf,
- host_buffer != orig_host_buffer
- ? MAXPACKET : 1024, (u_char **) &host_buffer);
+ n = __libc_res_nsearch (&_res, name, C_IN, T_A, host_buffer.buf->buf,
+ host_buffer.buf != orig_host_buffer
+ ? MAXPACKET : 1024, (u_char **) &host_buffer.ptr);
if (n < 0)
{
- if (host_buffer != orig_host_buffer)
- free (host_buffer);
+ if (host_buffer.buf != orig_host_buffer)
+ free (host_buffer.buf);
return status;
}
@@ -205,10 +210,10 @@ _nss_dns_gethostbyname2_r (const char *n
result->h_length = INADDRSZ;;
}
- status = getanswer_r (host_buffer, n, name, type, result, buffer, buflen,
+ status = getanswer_r (host_buffer.buf, n, name, type, result, buffer, buflen,
errnop, h_errnop, map);
- if (host_buffer != orig_host_buffer)
- free (host_buffer);
+ if (host_buffer.buf != orig_host_buffer)
+ free (host_buffer.buf);
return status;
}
@@ -247,7 +252,12 @@ _nss_dns_gethostbyaddr_r (const void *ad
char *h_addr_ptrs[MAX_NR_ADDRS + 1];
char linebuffer[0];
} *host_data = (struct host_data *) buffer;
- querybuf *host_buffer, *orig_host_buffer;
+ union
+ {
+ querybuf *buf;
+ void *ptr;
+ } host_buffer;
+ querybuf *orig_host_buffer;
char qbuf[MAXDNAME+1], *qp = NULL;
size_t size;
int n, status;
@@ -305,30 +315,30 @@ _nss_dns_gethostbyaddr_r (const void *ad
break;
}
- host_buffer = orig_host_buffer = (querybuf *) alloca (1024);
+ host_buffer.buf = orig_host_buffer = (querybuf *) alloca (1024);
- n = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, host_buffer->buf,
- 1024, (u_char **) &host_buffer);
+ n = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, host_buffer.buf->buf,
+ 1024, (u_char **) &host_buffer.ptr);
if (n < 0 && af == AF_INET6)
{
strcpy (qp, "ip6.int");
- n = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, host_buffer->buf,
- host_buffer != orig_host_buffer
- ? MAXPACKET : 1024, (u_char **) &host_buffer);
+ n = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, host_buffer.buf->buf,
+ host_buffer.buf != orig_host_buffer
+ ? MAXPACKET : 1024, (u_char **) &host_buffer.ptr);
}
if (n < 0)
{
*h_errnop = h_errno;
__set_errno (olderr);
- if (host_buffer != orig_host_buffer)
- free (host_buffer);
+ if (host_buffer.buf != orig_host_buffer)
+ free (host_buffer.buf);
return errno == ECONNREFUSED ? NSS_STATUS_UNAVAIL : NSS_STATUS_NOTFOUND;
}
- status = getanswer_r (host_buffer, n, qbuf, T_PTR, result, buffer, buflen,
+ status = getanswer_r (host_buffer.buf, n, qbuf, T_PTR, result, buffer, buflen,
errnop, h_errnop, 0 /* XXX */);
- if (host_buffer != orig_host_buffer)
- free (host_buffer);
+ if (host_buffer.buf != orig_host_buffer)
+ free (host_buffer.buf);
if (status != NSS_STATUS_SUCCESS)
{
*h_errnop = h_errno;
============================================================
Index: resolv/nss_dns/dns-network.c
--- resolv/nss_dns/dns-network.c 19 Nov 2002 06:40:16 -0000 1.17
+++ resolv/nss_dns/dns-network.c 28 Dec 2002 15:35:11 -0000
@@ -110,7 +110,12 @@ _nss_dns_getnetbyname_r (const char *nam
int *herrnop)
{
/* Return entry for network with NAME. */
- querybuf *net_buffer, *orig_net_buffer;
+ union
+ {
+ querybuf *buf;
+ void *ptr;
+ } net_buffer;
+ querybuf *orig_net_buffer;
int anslen;
char *qbuf;
enum nss_status status;
@@ -120,25 +125,25 @@ _nss_dns_getnetbyname_r (const char *nam
qbuf = strdupa (name);
- net_buffer = orig_net_buffer = (querybuf *) alloca (1024);
+ net_buffer.buf = orig_net_buffer = (querybuf *) alloca (1024);
- anslen = __libc_res_nsearch (&_res, qbuf, C_IN, T_PTR, net_buffer->buf,
- 1024, (u_char **) &net_buffer);
+ anslen = __libc_res_nsearch (&_res, qbuf, C_IN, T_PTR, net_buffer.buf->buf,
+ 1024, (u_char **) &net_buffer.buf);
if (anslen < 0)
{
/* Nothing found. */
*errnop = errno;
- if (net_buffer != orig_net_buffer)
- free (net_buffer);
+ if (net_buffer.buf != orig_net_buffer)
+ free (net_buffer.buf);
return (errno == ECONNREFUSED
|| errno == EPFNOSUPPORT
|| errno == EAFNOSUPPORT)
? NSS_STATUS_UNAVAIL : NSS_STATUS_NOTFOUND;
}
- status = getanswer_r (net_buffer, anslen, result, buffer, buflen, BYNAME);
- if (net_buffer != orig_net_buffer)
- free (net_buffer);
+ status = getanswer_r (net_buffer.buf, anslen, result, buffer, buflen, BYNAME);
+ if (net_buffer.buf != orig_net_buffer)
+ free (net_buffer.buf);
return status;
}
@@ -150,7 +155,12 @@ _nss_dns_getnetbyaddr_r (uint32_t net, i
{
/* Return entry for network with NAME. */
enum nss_status status;
- querybuf *net_buffer, *orig_net_buffer;
+ union
+ {
+ querybuf *buf;
+ void *ptr;
+ } net_buffer;
+ querybuf *orig_net_buffer;
unsigned int net_bytes[4];
char qbuf[MAXDNAME];
int cnt, anslen;
@@ -190,26 +200,26 @@ _nss_dns_getnetbyaddr_r (uint32_t net, i
break;
}
- net_buffer = orig_net_buffer = (querybuf *) alloca (1024);
+ net_buffer.buf = orig_net_buffer = (querybuf *) alloca (1024);
- anslen = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, net_buffer->buf,
- 1024, (u_char **) &net_buffer);
+ anslen = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, net_buffer.buf->buf,
+ 1024, (u_char **) &net_buffer.ptr);
if (anslen < 0)
{
/* Nothing found. */
int err = errno;
__set_errno (olderr);
- if (net_buffer != orig_net_buffer)
- free (net_buffer);
+ if (net_buffer.buf != orig_net_buffer)
+ free (net_buffer.buf);
return (err == ECONNREFUSED
|| err == EPFNOSUPPORT
|| err == EAFNOSUPPORT)
? NSS_STATUS_UNAVAIL : NSS_STATUS_NOTFOUND;
}
- status = getanswer_r (net_buffer, anslen, result, buffer, buflen, BYADDR);
- if (net_buffer != orig_net_buffer)
- free (net_buffer);
+ status = getanswer_r (net_buffer.buf, anslen, result, buffer, buflen, BYADDR);
+ if (net_buffer.buf != orig_net_buffer)
+ free (net_buffer.buf);
if (status == NSS_STATUS_SUCCESS)
{
/* Strip trailing zeros. */
--
Andreas Jaeger
SuSE Labs aj@suse.de
private aj@arthur.inka.de
http://www.suse.de/~aj
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Get rid of some type-punning warnings
2002-12-28 7:43 Get rid of some type-punning warnings Andreas Jaeger
@ 2002-12-28 13:06 ` Ulrich Drepper
2002-12-30 5:38 ` Andreas Jaeger
0 siblings, 1 reply; 3+ messages in thread
From: Ulrich Drepper @ 2002-12-28 13:06 UTC (permalink / raw)
To: Andreas Jaeger; +Cc: GNU libc hacker
Andreas Jaeger wrote:
*buffer, size_t buflen, int *errnop,
> int *h_errnop)
> {
> - querybuf *host_buffer, *orig_host_buffer;
> + union
> + {
> + querybuf *buf;
> + void *ptr;
> + } host_buffer;
> + querybuf *orig_host_buffer;
Why do you use void* for ptr and not u_char*?
> - n = __libc_res_nsearch (&_res, name, C_IN, type, host_buffer->buf,
> - 1024, (u_char **) &host_buffer);
> + n = __libc_res_nsearch (&_res, name, C_IN, type, host_buffer.buf->buf,
> + 1024, (u_char **) &host_buffer.ptr);
In this situation you could simlpy use &host_buffer.ptr.
>
> ============================================================
> Index: resolv/nss_dns/dns-network.c
> --- resolv/nss_dns/dns-network.c 19 Nov 2002 06:40:16 -0000 1.17
> +++ resolv/nss_dns/dns-network.c 28 Dec 2002 15:35:11 -0000
> @@ -110,7 +110,12 @@ _nss_dns_getnetbyname_r (const char *nam
> int *herrnop)
> {
> /* Return entry for network with NAME. */
> - querybuf *net_buffer, *orig_net_buffer;
> + union
> + {
> + querybuf *buf;
> + void *ptr;
> + } net_buffer;
> + querybuf *orig_net_buffer;
> int anslen;
> char *qbuf;
> enum nss_status status;
> @@ -120,25 +125,25 @@ _nss_dns_getnetbyname_r (const char *nam
>
> qbuf = strdupa (name);
>
> - net_buffer = orig_net_buffer = (querybuf *) alloca (1024);
> + net_buffer.buf = orig_net_buffer = (querybuf *) alloca (1024);
>
> - anslen = __libc_res_nsearch (&_res, qbuf, C_IN, T_PTR, net_buffer->buf,
> - 1024, (u_char **) &net_buffer);
> + anslen = __libc_res_nsearch (&_res, qbuf, C_IN, T_PTR, net_buffer.buf->buf,
> + 1024, (u_char **) &net_buffer.buf);
And here you use &net_buffer.buf? I thought this is what you want to avoid?
Please check the patch again. Make sure that if uchar* can be used it
is used and remove all the casts.
--
--------------. ,-. 444 Castro Street
Ulrich Drepper \ ,-----------------' \ Mountain View, CA 94041 USA
Red Hat `--' drepper at redhat.com `---------------------------
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Get rid of some type-punning warnings
2002-12-28 13:06 ` Ulrich Drepper
@ 2002-12-30 5:38 ` Andreas Jaeger
0 siblings, 0 replies; 3+ messages in thread
From: Andreas Jaeger @ 2002-12-30 5:38 UTC (permalink / raw)
To: Ulrich Drepper; +Cc: GNU libc hacker
Ulrich Drepper <drepper@redhat.com> writes:
> Andreas Jaeger wrote:
> *buffer, size_t buflen, int *errnop,
>> int *h_errnop)
>> {
>> - querybuf *host_buffer, *orig_host_buffer;
>> + union
>> + {
>> + querybuf *buf;
>> + void *ptr;
>> + } host_buffer;
>> + querybuf *orig_host_buffer;
>
> Why do you use void* for ptr and not u_char*?
You're right, I've fixed this and changed also one more file.
Here's a new patch. Ok to commit now?
Andreas
2002-12-30 Andreas Jaeger <aj@suse.de>
* resolv/nss_dns/dns-host.c (_nss_dns_gethostbyaddr_r): Use union
type to avoid strict aliasing problem.
(_nss_dns_gethostbyname2_r): Likewise.
* resolv/nss_dns/dns-network.c (_nss_dns_getnetbyaddr_r): Likewise.
(_nss_dns_getnetbyname_r): Likewise.
* resolv/gethnamaddr.c (_gethtent): Likewise.
(gethostbyname2): Likewise.
============================================================
Index: resolv/gethnamaddr.c
--- resolv/gethnamaddr.c 17 Oct 2002 21:49:34 -0000 1.43
+++ resolv/gethnamaddr.c 30 Dec 2002 13:11:09 -0000
@@ -510,7 +510,12 @@ gethostbyname2(name, af)
const char *name;
int af;
{
- querybuf *buf, *origbuf;
+ union
+ {
+ querybuf *buf;
+ u_char *ptr;
+ } buf;
+ querybuf *origbuf;
register const char *cp;
char *bp;
int n, size, type, len;
@@ -616,20 +621,20 @@ gethostbyname2(name, af)
break;
}
- buf = origbuf = (querybuf *) alloca (1024);
+ buf.buf = origbuf = (querybuf *) alloca (1024);
- if ((n = __libc_res_nsearch(&_res, name, C_IN, type, buf->buf, 1024,
- (u_char **) &buf)) < 0) {
- if (buf != origbuf)
- free (buf);
+ if ((n = __libc_res_nsearch(&_res, name, C_IN, type, buf.buf->buf, 1024,
+ &buf.ptr)) < 0) {
+ if (buf.buf != origbuf)
+ free (buf.buf);
dprintf("res_nsearch failed (%d)\n", n);
if (errno == ECONNREFUSED)
return (_gethtbyname2(name, af));
return (NULL);
}
- ret = getanswer(buf, n, name, type);
- if (buf != origbuf)
- free (buf);
+ ret = getanswer(buf.buf, n, name, type);
+ if (buf.buf != origbuf)
+ free (buf.buf);
return ret;
}
@@ -644,7 +649,12 @@ gethostbyaddr(addr, len, af)
static const u_char tunnelled[] = { 0,0, 0,0, 0,0, 0,0, 0,0, 0,0 };
int n;
socklen_t size;
- querybuf *buf, *orig_buf;
+ union
+ {
+ querybuf *buf;
+ u_char *ptr;
+ } buf;
+ querybuf *orig_buf;
register struct hostent *hp;
char qbuf[MAXDNAME+1], *qp = NULL;
#ifdef SUNSECURITY
@@ -706,27 +716,27 @@ gethostbyaddr(addr, len, af)
abort();
}
- buf = orig_buf = (querybuf *) alloca (1024);
+ buf.buf = orig_buf = (querybuf *) alloca (1024);
- n = __libc_res_nquery(&_res, qbuf, C_IN, T_PTR, buf->buf, 1024,
- (u_char **) &buf);
+ n = __libc_res_nquery(&_res, qbuf, C_IN, T_PTR, buf.buf->buf, 1024,
+ &buf.ptr);
if (n < 0 && af == AF_INET6) {
strcpy(qp, "ip6.int");
- n = __libc_res_nquery(&_res, qbuf, C_IN, T_PTR, buf->buf,
- buf != orig_buf ? MAXPACKET : 1024,
- (u_char **) &buf);
+ n = __libc_res_nquery(&_res, qbuf, C_IN, T_PTR, buf.buf->buf,
+ buf.buf != orig_buf ? MAXPACKET : 1024,
+ &buf.ptr);
}
if (n < 0) {
- if (buf != orig_buf)
- free (buf);
+ if (buf.buf != orig_buf)
+ free (buf.buf);
dprintf("res_nquery failed (%d)\n", n);
if (errno == ECONNREFUSED)
return (_gethtbyaddr(addr, len, af));
return (NULL);
}
- hp = getanswer(buf, n, qbuf, T_PTR);
- if (buf != orig_buf)
- free (buf);
+ hp = getanswer(buf.buf, n, qbuf, T_PTR);
+ if (buf.buf != orig_buf)
+ free (buf.buf);
if (!hp)
return (NULL); /* h_errno was set by getanswer() */
#ifdef SUNSECURITY
============================================================
Index: resolv/nss_dns/dns-network.c
--- resolv/nss_dns/dns-network.c 19 Nov 2002 06:40:16 -0000 1.17
+++ resolv/nss_dns/dns-network.c 30 Dec 2002 13:11:09 -0000
@@ -110,7 +110,12 @@ _nss_dns_getnetbyname_r (const char *nam
int *herrnop)
{
/* Return entry for network with NAME. */
- querybuf *net_buffer, *orig_net_buffer;
+ union
+ {
+ querybuf *buf;
+ u_char *ptr;
+ } net_buffer;
+ querybuf *orig_net_buffer;
int anslen;
char *qbuf;
enum nss_status status;
@@ -120,25 +125,25 @@ _nss_dns_getnetbyname_r (const char *nam
qbuf = strdupa (name);
- net_buffer = orig_net_buffer = (querybuf *) alloca (1024);
+ net_buffer.buf = orig_net_buffer = (querybuf *) alloca (1024);
- anslen = __libc_res_nsearch (&_res, qbuf, C_IN, T_PTR, net_buffer->buf,
- 1024, (u_char **) &net_buffer);
+ anslen = __libc_res_nsearch (&_res, qbuf, C_IN, T_PTR, net_buffer.buf->buf,
+ 1024, &net_buffer.ptr);
if (anslen < 0)
{
/* Nothing found. */
*errnop = errno;
- if (net_buffer != orig_net_buffer)
- free (net_buffer);
+ if (net_buffer.buf != orig_net_buffer)
+ free (net_buffer.buf);
return (errno == ECONNREFUSED
|| errno == EPFNOSUPPORT
|| errno == EAFNOSUPPORT)
? NSS_STATUS_UNAVAIL : NSS_STATUS_NOTFOUND;
}
- status = getanswer_r (net_buffer, anslen, result, buffer, buflen, BYNAME);
- if (net_buffer != orig_net_buffer)
- free (net_buffer);
+ status = getanswer_r (net_buffer.buf, anslen, result, buffer, buflen, BYNAME);
+ if (net_buffer.buf != orig_net_buffer)
+ free (net_buffer.buf);
return status;
}
@@ -150,7 +155,12 @@ _nss_dns_getnetbyaddr_r (uint32_t net, i
{
/* Return entry for network with NAME. */
enum nss_status status;
- querybuf *net_buffer, *orig_net_buffer;
+ union
+ {
+ querybuf *buf;
+ u_char *ptr;
+ } net_buffer;
+ querybuf *orig_net_buffer;
unsigned int net_bytes[4];
char qbuf[MAXDNAME];
int cnt, anslen;
@@ -190,26 +200,26 @@ _nss_dns_getnetbyaddr_r (uint32_t net, i
break;
}
- net_buffer = orig_net_buffer = (querybuf *) alloca (1024);
+ net_buffer.buf = orig_net_buffer = (querybuf *) alloca (1024);
- anslen = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, net_buffer->buf,
- 1024, (u_char **) &net_buffer);
+ anslen = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, net_buffer.buf->buf,
+ 1024, &net_buffer.ptr);
if (anslen < 0)
{
/* Nothing found. */
int err = errno;
__set_errno (olderr);
- if (net_buffer != orig_net_buffer)
- free (net_buffer);
+ if (net_buffer.buf != orig_net_buffer)
+ free (net_buffer.buf);
return (err == ECONNREFUSED
|| err == EPFNOSUPPORT
|| err == EAFNOSUPPORT)
? NSS_STATUS_UNAVAIL : NSS_STATUS_NOTFOUND;
}
- status = getanswer_r (net_buffer, anslen, result, buffer, buflen, BYADDR);
- if (net_buffer != orig_net_buffer)
- free (net_buffer);
+ status = getanswer_r (net_buffer.buf, anslen, result, buffer, buflen, BYADDR);
+ if (net_buffer.buf != orig_net_buffer)
+ free (net_buffer.buf);
if (status == NSS_STATUS_SUCCESS)
{
/* Strip trailing zeros. */
============================================================
Index: resolv/nss_dns/dns-host.c
--- resolv/nss_dns/dns-host.c 17 Oct 2002 21:48:48 -0000 1.36
+++ resolv/nss_dns/dns-host.c 30 Dec 2002 13:11:09 -0000
@@ -132,7 +132,12 @@ _nss_dns_gethostbyname2_r (const char *n
char *buffer, size_t buflen, int *errnop,
int *h_errnop)
{
- querybuf *host_buffer, *orig_host_buffer;
+ union
+ {
+ querybuf *buf;
+ u_char *ptr;
+ } host_buffer;
+ querybuf *orig_host_buffer;
char tmp[NS_MAXDNAME];
int size, type, n;
const char *cp;
@@ -170,10 +175,10 @@ _nss_dns_gethostbyname2_r (const char *n
&& (cp = res_hostalias (&_res, name, tmp, sizeof (tmp))) != NULL)
name = cp;
- host_buffer = orig_host_buffer = (querybuf *) alloca (1024);
+ host_buffer.buf = orig_host_buffer = (querybuf *) alloca (1024);
- n = __libc_res_nsearch (&_res, name, C_IN, type, host_buffer->buf,
- 1024, (u_char **) &host_buffer);
+ n = __libc_res_nsearch (&_res, name, C_IN, type, host_buffer.buf->buf,
+ 1024, &host_buffer.ptr);
if (n < 0)
{
enum nss_status status = (errno == ECONNREFUSED
@@ -188,14 +193,14 @@ _nss_dns_gethostbyname2_r (const char *n
by having the RES_USE_INET6 bit in _res.options set, we try
another lookup. */
if (af == AF_INET6 && (_res.options & RES_USE_INET6))
- n = __libc_res_nsearch (&_res, name, C_IN, T_A, host_buffer->buf,
- host_buffer != orig_host_buffer
- ? MAXPACKET : 1024, (u_char **) &host_buffer);
+ n = __libc_res_nsearch (&_res, name, C_IN, T_A, host_buffer.buf->buf,
+ host_buffer.buf != orig_host_buffer
+ ? MAXPACKET : 1024, &host_buffer.ptr);
if (n < 0)
{
- if (host_buffer != orig_host_buffer)
- free (host_buffer);
+ if (host_buffer.buf != orig_host_buffer)
+ free (host_buffer.buf);
return status;
}
@@ -205,10 +210,10 @@ _nss_dns_gethostbyname2_r (const char *n
result->h_length = INADDRSZ;;
}
- status = getanswer_r (host_buffer, n, name, type, result, buffer, buflen,
+ status = getanswer_r (host_buffer.buf, n, name, type, result, buffer, buflen,
errnop, h_errnop, map);
- if (host_buffer != orig_host_buffer)
- free (host_buffer);
+ if (host_buffer.buf != orig_host_buffer)
+ free (host_buffer.buf);
return status;
}
@@ -247,7 +252,12 @@ _nss_dns_gethostbyaddr_r (const void *ad
char *h_addr_ptrs[MAX_NR_ADDRS + 1];
char linebuffer[0];
} *host_data = (struct host_data *) buffer;
- querybuf *host_buffer, *orig_host_buffer;
+ union
+ {
+ querybuf *buf;
+ u_char *ptr;
+ } host_buffer;
+ querybuf *orig_host_buffer;
char qbuf[MAXDNAME+1], *qp = NULL;
size_t size;
int n, status;
@@ -305,30 +315,30 @@ _nss_dns_gethostbyaddr_r (const void *ad
break;
}
- host_buffer = orig_host_buffer = (querybuf *) alloca (1024);
+ host_buffer.buf = orig_host_buffer = (querybuf *) alloca (1024);
- n = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, host_buffer->buf,
- 1024, (u_char **) &host_buffer);
+ n = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, host_buffer.buf->buf,
+ 1024, &host_buffer.ptr);
if (n < 0 && af == AF_INET6)
{
strcpy (qp, "ip6.int");
- n = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, host_buffer->buf,
- host_buffer != orig_host_buffer
- ? MAXPACKET : 1024, (u_char **) &host_buffer);
+ n = __libc_res_nquery (&_res, qbuf, C_IN, T_PTR, host_buffer.buf->buf,
+ host_buffer.buf != orig_host_buffer
+ ? MAXPACKET : 1024, &host_buffer.ptr);
}
if (n < 0)
{
*h_errnop = h_errno;
__set_errno (olderr);
- if (host_buffer != orig_host_buffer)
- free (host_buffer);
+ if (host_buffer.buf != orig_host_buffer)
+ free (host_buffer.buf);
return errno == ECONNREFUSED ? NSS_STATUS_UNAVAIL : NSS_STATUS_NOTFOUND;
}
- status = getanswer_r (host_buffer, n, qbuf, T_PTR, result, buffer, buflen,
+ status = getanswer_r (host_buffer.buf, n, qbuf, T_PTR, result, buffer, buflen,
errnop, h_errnop, 0 /* XXX */);
- if (host_buffer != orig_host_buffer)
- free (host_buffer);
+ if (host_buffer.buf != orig_host_buffer)
+ free (host_buffer.buf);
if (status != NSS_STATUS_SUCCESS)
{
*h_errnop = h_errno;
--
Andreas Jaeger
SuSE Labs aj@suse.de
private aj@arthur.inka.de
http://www.suse.de/~aj
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2002-12-30 13:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-12-28 7:43 Get rid of some type-punning warnings Andreas Jaeger
2002-12-28 13:06 ` Ulrich Drepper
2002-12-30 5:38 ` Andreas Jaeger
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).