libbb: make INET[6]_rresolve use sockaddr2{host,dotted}_noport

function                                             old     new   delta
INET_rresolve                                        274     214     -60
INET6_rresolve                                       168      85     -83

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2015-02-03 12:07:40 +01:00
parent f7f70bf1b3
commit 8e74adab01

View File

@ -32,14 +32,12 @@ int FAST_FUNC INET_resolve(const char *name, struct sockaddr_in *s_in, int hostf
return 0; return 0;
} }
/* If we expect this to be a hostname, try hostname database first */ /* If we expect this to be a hostname, try hostname database first */
if (hostfirst) {
#ifdef DEBUG #ifdef DEBUG
if (hostfirst) {
bb_error_msg("gethostbyname(%s)", name); bb_error_msg("gethostbyname(%s)", name);
}
#endif #endif
if (hostfirst) {
hp = gethostbyname(name); hp = gethostbyname(name);
if (hp != NULL) { if (hp) {
memcpy(&s_in->sin_addr, hp->h_addr_list[0], memcpy(&s_in->sin_addr, hp->h_addr_list[0],
sizeof(struct in_addr)); sizeof(struct in_addr));
return 0; return 0;
@ -51,7 +49,7 @@ int FAST_FUNC INET_resolve(const char *name, struct sockaddr_in *s_in, int hostf
bb_error_msg("getnetbyname(%s)", name); bb_error_msg("getnetbyname(%s)", name);
#endif #endif
np = getnetbyname(name); np = getnetbyname(name);
if (np != NULL) { if (np) {
s_in->sin_addr.s_addr = htonl(np->n_net); s_in->sin_addr.s_addr = htonl(np->n_net);
return 1; return 1;
} }
@ -66,7 +64,7 @@ int FAST_FUNC INET_resolve(const char *name, struct sockaddr_in *s_in, int hostf
bb_error_msg("gethostbyname(%s)", name); bb_error_msg("gethostbyname(%s)", name);
#endif #endif
hp = gethostbyname(name); hp = gethostbyname(name);
if (hp == NULL) { if (!hp) {
return -1; return -1;
} }
memcpy(&s_in->sin_addr, hp->h_addr_list[0], sizeof(struct in_addr)); memcpy(&s_in->sin_addr, hp->h_addr_list[0], sizeof(struct in_addr));
@ -74,7 +72,7 @@ int FAST_FUNC INET_resolve(const char *name, struct sockaddr_in *s_in, int hostf
} }
/* numeric: & 0x8000: default instead of *, /* numeric: & 0x8000: "default" instead of "*",
* & 0x4000: host instead of net, * & 0x4000: host instead of net,
* & 0x0fff: don't resolve * & 0x0fff: don't resolve
*/ */
@ -83,16 +81,16 @@ char* FAST_FUNC INET_rresolve(struct sockaddr_in *s_in, int numeric, uint32_t ne
/* addr-to-name cache */ /* addr-to-name cache */
struct addr { struct addr {
struct addr *next; struct addr *next;
struct sockaddr_in addr; uint32_t nip;
int host; smallint is_host;
char name[1]; char name[1];
}; };
static struct addr *cache = NULL; static struct addr *cache = NULL;
struct addr *pn; struct addr *pn;
char *name; char *name;
uint32_t ad, host_ad; uint32_t nip;
int host = 0; smallint is_host;
if (s_in->sin_family != AF_INET) { if (s_in->sin_family != AF_INET) {
#ifdef DEBUG #ifdef DEBUG
@ -102,61 +100,57 @@ char* FAST_FUNC INET_rresolve(struct sockaddr_in *s_in, int numeric, uint32_t ne
errno = EAFNOSUPPORT; errno = EAFNOSUPPORT;
return NULL; return NULL;
} }
ad = s_in->sin_addr.s_addr; nip = s_in->sin_addr.s_addr;
#ifdef DEBUG #ifdef DEBUG
bb_error_msg("rresolve: %08x, mask %08x, num %08x", (unsigned)ad, netmask, numeric); bb_error_msg("rresolve: %08x mask:%08x num:%08x", (unsigned)nip, netmask, numeric);
#endif #endif
if (ad == INADDR_ANY) {
if ((numeric & 0x0FFF) == 0) {
if (numeric & 0x8000)
return xstrdup("default");
return xstrdup("*");
}
}
if (numeric & 0x0FFF) if (numeric & 0x0FFF)
return xstrdup(inet_ntoa(s_in->sin_addr)); return xmalloc_sockaddr2dotted_noport((void*)s_in);
if (nip == INADDR_ANY) {
if (numeric & 0x8000)
return xstrdup("default");
return xstrdup("*");
}
is_host = ((nip & (~netmask)) != 0 || (numeric & 0x4000));
if ((ad & (~netmask)) != 0 || (numeric & 0x4000))
host = 1;
pn = cache; pn = cache;
while (pn) { while (pn) {
if (pn->addr.sin_addr.s_addr == ad && pn->host == host) { if (pn->nip == nip && pn->is_host == is_host) {
#ifdef DEBUG #ifdef DEBUG
bb_error_msg("rresolve: found %s %08x in cache", bb_error_msg("rresolve: found %s %08x in cache",
(host ? "host" : "net"), (unsigned)ad); (is_host ? "host" : "net"), (unsigned)nip);
#endif #endif
return xstrdup(pn->name); return xstrdup(pn->name);
} }
pn = pn->next; pn = pn->next;
} }
host_ad = ntohl(ad);
name = NULL; name = NULL;
if (host) { if (is_host) {
struct hostent *ent;
#ifdef DEBUG #ifdef DEBUG
bb_error_msg("gethostbyaddr (%08x)", (unsigned)ad); bb_error_msg("sockaddr2host_noport(%08x)", (unsigned)nip);
#endif #endif
ent = gethostbyaddr((char *) &ad, 4, AF_INET); name = xmalloc_sockaddr2host_noport((void*)s_in);
if (ent)
name = xstrdup(ent->h_name);
} else if (ENABLE_FEATURE_ETC_NETWORKS) { } else if (ENABLE_FEATURE_ETC_NETWORKS) {
struct netent *np; struct netent *np;
#ifdef DEBUG #ifdef DEBUG
bb_error_msg("getnetbyaddr (%08x)", (unsigned)host_ad); bb_error_msg("getnetbyaddr(%08x)", (unsigned)ntohl(nip));
#endif #endif
np = getnetbyaddr(host_ad, AF_INET); np = getnetbyaddr(ntohl(nip), AF_INET);
if (np) if (np)
name = xstrdup(np->n_name); name = xstrdup(np->n_name);
} }
if (!name) if (!name)
name = xstrdup(inet_ntoa(s_in->sin_addr)); name = xmalloc_sockaddr2dotted_noport((void*)s_in);
pn = xmalloc(sizeof(*pn) + strlen(name)); /* no '+ 1', it's already accounted for */ pn = xmalloc(sizeof(*pn) + strlen(name)); /* no '+ 1', it's already accounted for */
pn->next = cache; pn->next = cache;
pn->addr = *s_in; pn->nip = nip;
pn->host = host; pn->is_host = is_host;
strcpy(pn->name, name); strcpy(pn->name, name);
cache = pn; cache = pn;
return name; return name;
} }
@ -188,9 +182,6 @@ int FAST_FUNC INET6_resolve(const char *name, struct sockaddr_in6 *sin6)
char* FAST_FUNC INET6_rresolve(struct sockaddr_in6 *sin6, int numeric) char* FAST_FUNC INET6_rresolve(struct sockaddr_in6 *sin6, int numeric)
{ {
char name[128];
int s;
if (sin6->sin6_family != AF_INET6) { if (sin6->sin6_family != AF_INET6) {
#ifdef DEBUG #ifdef DEBUG
bb_error_msg("rresolve: unsupported address family %d!", bb_error_msg("rresolve: unsupported address family %d!",
@ -200,8 +191,7 @@ char* FAST_FUNC INET6_rresolve(struct sockaddr_in6 *sin6, int numeric)
return NULL; return NULL;
} }
if (numeric & 0x7FFF) { if (numeric & 0x7FFF) {
inet_ntop(AF_INET6, &sin6->sin6_addr, name, sizeof(name)); return xmalloc_sockaddr2dotted_noport((void*)sin6);
return xstrdup(name);
} }
if (IN6_IS_ADDR_UNSPECIFIED(&sin6->sin6_addr)) { if (IN6_IS_ADDR_UNSPECIFIED(&sin6->sin6_addr)) {
if (numeric & 0x8000) if (numeric & 0x8000)
@ -209,15 +199,7 @@ char* FAST_FUNC INET6_rresolve(struct sockaddr_in6 *sin6, int numeric)
return xstrdup("*"); return xstrdup("*");
} }
s = getnameinfo((struct sockaddr *) sin6, sizeof(*sin6), return xmalloc_sockaddr2host_noport((void*)sin6);
name, sizeof(name),
/*serv,servlen:*/ NULL, 0,
0);
if (s != 0) {
bb_error_msg("getnameinfo failed");
return NULL;
}
return xstrdup(name);
} }
#endif /* CONFIG_FEATURE_IPV6 */ #endif /* CONFIG_FEATURE_IPV6 */