[git commit] resolv: simple optimizations and style/readability fixes
Denys Vlasenko
vda.linux at googlemail.com
Sat Sep 10 01:01:33 UTC 2011
commit: http://git.uclibc.org/uClibc/commit/?id=021b0aa3c497860ea964b96b2cec8f1290846891
branch: http://git.uclibc.org/uClibc/commit/?id=refs/heads/master
text data bss dec hex filename
- 2915 0 0 2915 b63 libc/inet/ns_name_old.o
+ 2656 0 0 2656 a60 libc/inet/ns_name.o
- 958 0 0 958 3be libc/inet/ns_parse_old.o
+ 898 0 0 898 382 libc/inet/ns_parse.o
Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
include/arpa/nameser.h | 4 -
libc/inet/resolv.c | 225 +++++++++++++++++++++++-------------------------
2 files changed, 107 insertions(+), 122 deletions(-)
diff --git a/include/arpa/nameser.h b/include/arpa/nameser.h
index 1534249..dfab5b4 100644
--- a/include/arpa/nameser.h
+++ b/include/arpa/nameser.h
@@ -118,10 +118,6 @@ typedef struct __ns_msg {
const u_char *_ptr;
} ns_msg;
-/* Private data structure - do not use from outside library. */
-struct _ns_flagdata { int mask, shift; };
-extern const struct _ns_flagdata _ns_flagdata[];
-
/* Accessor macros - this is part of the public interface. */
#define ns_msg_id(handle) ((handle)._id + 0)
#define ns_msg_base(handle) ((handle)._msg + 0)
diff --git a/libc/inet/resolv.c b/libc/inet/resolv.c
index 34dfbfa..0fd28ee 100644
--- a/libc/inet/resolv.c
+++ b/libc/inet/resolv.c
@@ -1582,12 +1582,12 @@ int attribute_hidden __dns_lookup(const char *name,
parser_t * __open_etc_hosts(void)
{
- parser_t * parser;
- if ((parser = config_open("/etc/hosts")) == NULL) {
+ parser_t *parser;
+ parser = config_open("/etc/hosts");
#ifdef FALLBACK_TO_CONFIG_RESOLVCONF
+ if (parser == NULL)
parser = config_open("/etc/config/hosts");
#endif
- }
return parser;
}
@@ -2739,8 +2739,6 @@ libc_hidden_def(ns_name_uncompress)
*/
int ns_name_ntop(const u_char *src, char *dst, size_t dstsiz)
{
- static const char digits[] = "0123456789";
-
const u_char *cp;
char *dn, *eom;
u_char c;
@@ -2782,9 +2780,10 @@ int ns_name_ntop(const u_char *src, char *dst, size_t dstsiz)
return -1;
}
*dn++ = '\\';
- *dn++ = digits[c / 100];
- *dn++ = digits[(c % 100) / 10];
- *dn++ = digits[c % 10];
+ *dn++ = "0123456789"[c / 100];
+ c = c % 100;
+ *dn++ = "0123456789"[c / 10];
+ *dn++ = "0123456789"[c % 10];
} else {
if (dn >= eom) {
__set_errno(EMSGSIZE);
@@ -2810,27 +2809,6 @@ int ns_name_ntop(const u_char *src, char *dst, size_t dstsiz)
}
libc_hidden_def(ns_name_ntop)
-static const char digits[] = "0123456789";
-
-static const char digitvalue[256] = {
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, /*16*/
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, /*32*/
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, /*48*/
- 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, -1, -1, -1, -1, -1, -1, /*64*/
- -1, 10, 11, 12, 13, 14, 15, -1, -1, -1, -1, -1, -1, -1, -1, -1, /*80*/
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, /*96*/
- -1, 10, 11, 12, 13, 14, 15, -1, -1, -1, -1, -1, -1, -1, -1, -1, /*112*/
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, /*128*/
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
- -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, /*256*/
-};
-
static int encode_bitstring(const char **bp, const char *end,
unsigned char **labelp,
unsigned char ** dst,
@@ -2839,32 +2817,33 @@ static int encode_bitstring(const char **bp, const char *end,
int afterslash = 0;
const char *cp = *bp;
unsigned char *tp;
- char c;
const char *beg_blen;
- char *end_blen = NULL;
int value = 0, count = 0, tbcount = 0, blen = 0;
- beg_blen = end_blen = NULL;
+ beg_blen = NULL;
/* a bitstring must contain at least 2 characters */
if (end - cp < 2)
- return (EINVAL);
+ return EINVAL;
/* XXX: currently, only hex strings are supported */
if (*cp++ != 'x')
- return (EINVAL);
- if (!isxdigit((*cp) & 0xff)) /*%< reject '\[x/BLEN]' */
- return (EINVAL);
+ return EINVAL;
+ if (!isxdigit((unsigned char) *cp)) /*%< reject '\[x/BLEN]' */
+ return EINVAL;
for (tp = *dst + 1; cp < end && tp < eom; cp++) {
- switch((c = *cp)) {
+ unsigned char c = *cp;
+
+ switch (c) {
case ']': /*%< end of the bitstring */
if (afterslash) {
+ char *end_blen;
if (beg_blen == NULL)
- return (EINVAL);
+ return EINVAL;
blen = (int)strtol(beg_blen, &end_blen, 10);
if (*end_blen != ']')
- return (EINVAL);
+ return EINVAL;
}
if (count)
*tp++ = ((value << 4) & 0xff);
@@ -2875,25 +2854,29 @@ static int encode_bitstring(const char **bp, const char *end,
break;
default:
if (afterslash) {
- if (!isdigit(c&0xff))
- return (EINVAL);
+ if (!__isdigit_char(c))
+ return EINVAL;
if (beg_blen == NULL) {
-
if (c == '0') {
/* blen never begings with 0 */
- return (EINVAL);
+ return EINVAL;
}
beg_blen = cp;
}
} else {
- if (!isxdigit(c&0xff))
- return (EINVAL);
+ if (!__isdigit_char(c)) {
+ c = c | 0x20; /* lowercase */
+ c = c - 'a';
+ if (c > 5) /* not a-f? */
+ return EINVAL;
+ c += 10 + '0';
+ }
value <<= 4;
- value += digitvalue[(int)c];
+ value += (c - '0');
count += 4;
tbcount += 4;
if (tbcount > 256)
- return (EINVAL);
+ return EINVAL;
if (count == 8) {
*tp++ = value;
count = 0;
@@ -2904,7 +2887,7 @@ static int encode_bitstring(const char **bp, const char *end,
}
done:
if (cp >= end || tp >= eom)
- return (EMSGSIZE);
+ return EMSGSIZE;
/*
* bit length validation:
@@ -2918,10 +2901,10 @@ static int encode_bitstring(const char **bp, const char *end,
int traillen;
if (((blen + 3) & ~3) != tbcount)
- return (EINVAL);
+ return EINVAL;
traillen = tbcount - blen; /*%< between 0 and 3 */
if (((value << (8 - traillen)) & 0xff) != 0)
- return (EINVAL);
+ return EINVAL;
}
else
blen = tbcount;
@@ -2935,11 +2918,12 @@ static int encode_bitstring(const char **bp, const char *end,
*bp = cp;
*dst = tp;
- return (0);
+ return 0;
}
int ns_name_pton(const char *src, u_char *dst, size_t dstsiz)
{
+ static const char digits[] = "0123456789";
u_char *label, *bp, *eom;
int c, n, escaped, e = 0;
char *cp;
@@ -2952,44 +2936,47 @@ int ns_name_pton(const char *src, u_char *dst, size_t dstsiz)
while ((c = *src++) != 0) {
if (escaped) {
if (c == '[') { /*%< start a bit string label */
- if ((cp = strchr(src, ']')) == NULL) {
+ cp = strchr(src, ']');
+ if (cp == NULL) {
errno = EINVAL; /*%< ??? */
- return (-1);
+ return -1;
}
- if ((e = encode_bitstring(&src, cp + 2,
- &label, &bp, eom))
- != 0) {
+ e = encode_bitstring(&src, cp + 2,
+ &label, &bp, eom);
+ if (e != 0) {
errno = e;
- return (-1);
+ return -1;
}
escaped = 0;
label = bp++;
- if ((c = *src++) == 0)
+ c = *src++;
+ if (c == '\0')
goto done;
- else if (c != '.') {
+ if (c != '.') {
errno = EINVAL;
- return (-1);
+ return -1;
}
continue;
}
- else if ((cp = strchr(digits, c)) != NULL) {
+ cp = strchr(digits, c);
+ if (cp != NULL) {
n = (cp - digits) * 100;
- if ((c = *src++) == 0 ||
- (cp = strchr(digits, c)) == NULL) {
- errno = EMSGSIZE;
- return (-1);
- }
+ c = *src++;
+ if (c == '\0')
+ goto ret_EMSGSIZE;
+ cp = strchr(digits, c);
+ if (cp == NULL)
+ goto ret_EMSGSIZE;
n += (cp - digits) * 10;
- if ((c = *src++) == 0 ||
- (cp = strchr(digits, c)) == NULL) {
- errno = EMSGSIZE;
- return (-1);
- }
+ c = *src++;
+ if (c == '\0')
+ goto ret_EMSGSIZE;
+ cp = strchr(digits, c);
+ if (cp == NULL)
+ goto ret_EMSGSIZE;
n += (cp - digits);
- if (n > 255) {
- errno = EMSGSIZE;
- return (-1);
- }
+ if (n > 255)
+ goto ret_EMSGSIZE;
c = n;
}
escaped = 0;
@@ -2999,67 +2986,61 @@ int ns_name_pton(const char *src, u_char *dst, size_t dstsiz)
} else if (c == '.') {
c = (bp - label - 1);
if ((c & NS_CMPRSFLGS) != 0) { /*%< Label too big. */
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
if (label >= eom) {
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
*label = c;
/* Fully qualified ? */
if (*src == '\0') {
if (c != 0) {
if (bp >= eom) {
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
*bp++ = '\0';
}
if ((bp - dst) > MAXCDNAME) {
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
- return (1);
+ return 1;
}
if (c == 0 || *src == '.') {
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
label = bp++;
continue;
}
if (bp >= eom) {
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
*bp++ = (u_char)c;
}
c = (bp - label - 1);
if ((c & NS_CMPRSFLGS) != 0) { /*%< Label too big. */
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
- done:
+ done:
if (label >= eom) {
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
*label = c;
if (c != 0) {
if (bp >= eom) {
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
*bp++ = 0;
}
if ((bp - dst) > MAXCDNAME) { /*%< src too big */
- errno = EMSGSIZE;
- return (-1);
+ goto ret_EMSGSIZE;
}
- return (0);
+ return 0;
+
+ ret_EMSGSIZE:
+ errno = EMSGSIZE;
+ return -1;
}
libc_hidden_def(ns_name_pton)
@@ -3140,7 +3121,7 @@ libc_hidden_def(ns_name_unpack)
static int labellen(const unsigned char *lp)
{
- int bitlen;
+ unsigned bitlen;
unsigned char l = *lp;
if ((l & NS_CMPRSFLGS) == NS_CMPRSFLGS) {
@@ -3150,7 +3131,8 @@ static int labellen(const unsigned char *lp)
if ((l & NS_CMPRSFLGS) == NS_TYPE_ELT) {
if (l == DNS_LABELTYPE_BITSTRING) {
- if ((bitlen = *(lp + 1)) == 0)
+ bitlen = lp[1];
+ if (bitlen == 0)
bitlen = 256;
return ((bitlen + 7 ) / 8 + 1);
}
@@ -3166,7 +3148,7 @@ static int mklower(int ch)
if (ch >= 0x41 && ch <= 0x5A)
return (ch + 0x20);
- return (ch);
+ return ch;
}
static int dn_find(const unsigned char *domain,
@@ -3245,7 +3227,8 @@ int ns_name_pack(const unsigned char *src,
lpp = cpp = NULL;
if (dnptrs != NULL) {
- if ((msg = *dnptrs++) != NULL) {
+ msg = *dnptrs++;
+ if (msg != NULL) {
for (cpp = dnptrs; *cpp != NULL; cpp++)
continue;
@@ -3266,7 +3249,8 @@ int ns_name_pack(const unsigned char *src,
return -1;
}
- if ((l0 = labellen(srcp)) < 0) {
+ l0 = labellen(srcp);
+ if (l0 < 0) {
errno = EINVAL;
return -1;
}
@@ -3365,7 +3349,8 @@ int ns_name_skip(const unsigned char **ptrptr,
cp += n;
continue;
case NS_TYPE_ELT: /*%< EDNS0 extended label */
- if ((l = labellen(cp - 1)) < 0) {
+ l = labellen(cp - 1);
+ if (l < 0) {
errno = EMSGSIZE; /*%< XXX */
return -1;
}
@@ -3546,7 +3531,7 @@ __res_vinit(res_state rp, int preinit)
}
static void
-__res_iclose(res_state statp, int free_addr)
+__res_iclose(void)
{
__UCLIBC_MUTEX_LOCK(__resolv_lock);
__close_nameservers();
@@ -3578,15 +3563,15 @@ __res_iclose(res_state statp, int free_addr)
void
res_nclose(res_state statp)
{
- __res_iclose(statp, 1);
+ __res_iclose();
}
#ifdef __UCLIBC_HAS_BSD_RES_CLOSE__
void res_close(void)
{
- __res_iclose(&_res, 0);
+ __res_iclose();
}
-#endif /* __UCLIBC_HAS_BSD_RES_CLOSE__ */
+#endif
/* This needs to be after the use of _res in res_init, above. */
#undef _res
@@ -3618,7 +3603,8 @@ struct __res_state *__resp = &_res;
#endif /* !__UCLIBC_HAS_THREADS__ */
static unsigned int
-res_randomid(void) {
+res_randomid(void)
+{
return 0xffff & getpid();
}
@@ -3649,18 +3635,18 @@ res_init(void)
__UCLIBC_MUTEX_LOCK(__resolv_lock);
if (!_res.retrans)
- _res.retrans = RES_TIMEOUT;
+ _res.retrans = RES_TIMEOUT;
if (!_res.retry)
- _res.retry = 4;
+ _res.retry = 4;
if (!(_res.options & RES_INIT))
- _res.options = RES_DEFAULT;
+ _res.options = RES_DEFAULT;
/*
* This one used to initialize implicitly to zero, so unless the app
* has set it to something in particular, we can randomize it now.
*/
if (!_res.id)
- _res.id = res_randomid();
+ _res.id = res_randomid();
__res_sync = res_sync_func;
__UCLIBC_MUTEX_UNLOCK(__resolv_lock);
@@ -4003,7 +3989,8 @@ void ns_put32(unsigned long src, unsigned char *dst)
#ifdef L_ns_parse
/* These need to be in the same order as the nres.h:ns_flag enum. */
-const struct _ns_flagdata _ns_flagdata[16] = {
+struct _ns_flagdata { unsigned short mask, shift; };
+static const struct _ns_flagdata _ns_flagdata[16] = {
{ 0x8000, 15 }, /*%< qr. */
{ 0x7800, 11 }, /*%< opcode. */
{ 0x0400, 10 }, /*%< aa. */
@@ -4150,7 +4137,7 @@ ns_parserr(ns_msg *handle, ns_sect section, int rrnum, ns_rr *rr)
rrnum - handle->_rrnum);
if (b < 0)
- return (-1);
+ return -1;
handle->_ptr += b;
handle->_rrnum = rrnum;
}
@@ -4159,7 +4146,7 @@ ns_parserr(ns_msg *handle, ns_sect section, int rrnum, ns_rr *rr)
b = dn_expand(handle->_msg, handle->_eom,
handle->_ptr, rr->name, NS_MAXDNAME);
if (b < 0)
- return (-1);
+ return -1;
handle->_ptr += b;
if (handle->_ptr + NS_INT16SZ + NS_INT16SZ > handle->_eom) {
errno = EMSGSIZE;
@@ -4191,7 +4178,8 @@ ns_parserr(ns_msg *handle, ns_sect section, int rrnum, ns_rr *rr)
return 0;
}
-int ns_msg_getflag(ns_msg handle, int flag) {
+int ns_msg_getflag(ns_msg handle, int flag)
+{
return ((handle)._flags & _ns_flagdata[flag].mask) >> _ns_flagdata[flag].shift;
}
#endif /* L_ns_parse */
@@ -4251,7 +4239,8 @@ int res_mkquery(int op, const char *dname, int class, int type,
if (ep - cp < QFIXEDSZ)
return -1;
- if ((n = dn_comp(dname, cp, ep - cp - QFIXEDSZ, dnptrs, lastdnptr)) < 0)
+ n = dn_comp(dname, cp, ep - cp - QFIXEDSZ, dnptrs, lastdnptr);
+ if (n < 0)
return -1;
cp += n;
--
1.7.3.4
More information about the uClibc-cvs
mailing list