diff options
| author | David 'Digit' Turner <digit@google.com> | 2010-03-08 15:13:04 -0800 |
|---|---|---|
| committer | David 'Digit' Turner <digit@google.com> | 2010-03-08 15:22:13 -0800 |
| commit | d378c68d74cb4fdac450650fe816c9d649c8edaf (patch) | |
| tree | 0d5c15b4606982f26009bdc5c572415ac9dd18d3 /libc/docs | |
| parent | d7ed1ae98249af8a58ce97784a29f048d77789a9 (diff) | |
| download | bionic-d378c68d74cb4fdac450650fe816c9d649c8edaf.zip bionic-d378c68d74cb4fdac450650fe816c9d649c8edaf.tar.gz bionic-d378c68d74cb4fdac450650fe816c9d649c8edaf.tar.bz2 | |
Fix spurious DNS lookups in the C library.
The problem was that the 'defdname' field of res_state structure
was not properly initialized in __res_vinit(). This field is used
to store the default domain name, which is normally build from
calling gethostname() (see line 549 of res_init.c).
Unfortunately, in the typical Android case, gethostname() returns
an error (the hostname is configured) and a random stack string is
used later to build the DNS search list (see lines 556+ in res_init.c)
For the sake of illustration, let's say the search list is set to
a random value like 'xWLK'.
The end result is that when trying to result an unknown domain name
(e.g. 'www.ptn'), the query fails then the resolver tries to make a
new query with the DNS search list path(s) appended (e.g. 'www.ptn.xWLK').
The patch simply initializes 'defdname' to an empty string to avoid
this when the net.dns.search system property is not set.
Also contains whitespace/formatting fixes
Diffstat (limited to 'libc/docs')
| -rw-r--r-- | libc/docs/CHANGES.TXT | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/libc/docs/CHANGES.TXT b/libc/docs/CHANGES.TXT index c053714..4f87547 100644 --- a/libc/docs/CHANGES.TXT +++ b/libc/docs/CHANGES.TXT @@ -76,6 +76,10 @@ Differences between current and Android 2.1: - <time.h>: Fix timezone management implementation to properly update 'tm_gmtoff' field in 'struct tm' structure. +- DNS: get rid of spurious random DNS queries when trying to resolve + an unknown domain name. Due to an initialization bug, a random DNS search + list was generated for each thread is net.dns.search is not defined. + ------------------------------------------------------------------------------- Differences between Android 2.1 and 2.0.1: |