diff options
| author | Jukka Rissanen <jukka.rissanen@linux.intel.com> | 2011-11-23 15:31:38 +0200 |
|---|---|---|
| committer | Samuel Ortiz <sameo@linux.intel.com> | 2011-11-24 01:21:43 +0100 |
| commit | a781b520ba389a0723161bababfbddc0ee2c8e41 (patch) | |
| tree | c403c7a1a060e771849bbdb34e9563b00999bf78 /src | |
| parent | 2aa4554d70915341eb0a3fdf7accd576ac5e9aa9 (diff) | |
| download | connman-a781b520ba389a0723161bababfbddc0ee2c8e41.tar.gz connman-a781b520ba389a0723161bababfbddc0ee2c8e41.tar.bz2 connman-a781b520ba389a0723161bababfbddc0ee2c8e41.zip | |
dnsproxy: Get rid of extra strlen() calls
Diffstat (limited to 'src')
| -rw-r--r-- | src/dnsproxy.c | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/src/dnsproxy.c b/src/dnsproxy.c index 1d7a2150..24aad910 100644 --- a/src/dnsproxy.c +++ b/src/dnsproxy.c @@ -279,6 +279,7 @@ static int append_query(unsigned char *buf, unsigned int size, { unsigned char *ptr = buf; char *offset; + int len; DBG("query %s domain %s", query, domain); @@ -288,11 +289,12 @@ static int append_query(unsigned char *buf, unsigned int size, tmp = strchr(offset, '.'); if (tmp == NULL) { - if (strlen(offset) == 0) + len = strlen(offset); + if (len == 0) break; - *ptr = strlen(offset); - memcpy(ptr + 1, offset, strlen(offset)); - ptr += strlen(offset) + 1; + *ptr = len; + memcpy(ptr + 1, offset, len); + ptr += len + 1; break; } @@ -309,11 +311,12 @@ static int append_query(unsigned char *buf, unsigned int size, tmp = strchr(offset, '.'); if (tmp == NULL) { - if (strlen(offset) == 0) + len = strlen(offset); + if (len == 0) break; - *ptr = strlen(offset); - memcpy(ptr + 1, offset, strlen(offset)); - ptr += strlen(offset) + 1; + *ptr = len; + memcpy(ptr + 1, offset, len); + ptr += len + 1; break; } |