History log of /system/netd/DnsProxyListener.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
0cdb680c7648b0f48e6513926cb0a06d290a5cbe 05-Jul-2012 Jurijs Oniscuks <jurijs.oniscuks@sonymobile.com> Fix race condition in pthread_create()

Do not send "&this->mThread" to pthread_create() as newly
created thread can "delete this" before pthread_create() saves
thread id. This would lead to heap corruption.

Change-Id: I9c4cd5e1002b85985514a36ae542778a783db6fb
/system/netd/DnsProxyListener.cpp
40e0c13d6b77a9260b6da4c9e11cca8a5f994ea2 09-Mar-2012 Selim Gurun <sgurun@google.com> We are not reverting this CL since it was a temporary fix and we just submitted the proper one.

Revert "Do not send extra data the client is not consuming."

This reverts commit 6217272e1a41fc9827250c675a0499d2091ec6ac
/system/netd/DnsProxyListener.cpp
6217272e1a41fc9827250c675a0499d2091ec6ac 09-Mar-2012 Selim Gurun <sgurun@google.com> Do not send extra data the client is not consuming.

Bug:6138617

This is to temporarily fix a problem in netd. In case of an error do
not send extra data that the client is not expecting. This causes a
netd crash.

Change-Id: Iadb375e326577857034c2a7d2e962e4242d4735f
/system/netd/DnsProxyListener.cpp
ddb34755fb54882b3ece8d4919593e26a2c1cfcb 28-Feb-2012 Selim Gurun <sgurun@google.com> Prevent potential stall on dns proxy operations.

Update wire protocol to return and process error code first. This
will make sure dns proxy operations do not stall when an internal
error happens.

Change-Id: I65595abb0638722ccd877f69057bd435ca06e524
/system/netd/DnsProxyListener.cpp
0e76b761a1514d5182675dd7b7d33725f62d6bc5 06-Jan-2012 Steve Block <steveblock@google.com> Rename (IF_)LOGW(_IF) to (IF_)ALOGW(_IF) DO NOT MERGE

See https://android-git.corp.google.com/g/157065

Bug: 5449033
Change-Id: Ic9a12b3bc322d1fa60de50725b5ca18341d64b91
/system/netd/DnsProxyListener.cpp
7b984e3f7e724f8a3547a707210319f3d479f261 20-Dec-2011 Steve Block <steveblock@google.com> Rename (IF_)LOGD(_IF) to (IF_)ALOGD(_IF) DO NOT MERGE

See https://android-git.corp.google.com/g/156016

Bug: 5449033
Change-Id: I85b946663d5a0e7606140282bf758c27b8c6f68f
/system/netd/DnsProxyListener.cpp
0475ae98cd8f45de815d67d1966edaf5997be9a9 27-Apr-2011 Brian Carlstrom <bdc@google.com> DNS Proxy should not hang on addresses containing null bytes

Two bugs here:

1.) The gethostbyaddr netd protocol was passing a binary address value
but the arguments are expected to be strings not containing
embedded nulls (or probably other special characters). This
happened reliably with IPv6 addresses which contained nulls. It
now expects an inet_ntop formatted address.

2.) Although the gethostbyaddr code detected something was wrong, it
wasn't doing any proper error handling, leaving clients hanging
waiting for a response. It now sends back an empty response so
that clients can continue.

A corresponding change will be necesssary in bionic for #1 for DNS
proxying to work, but given the fix for #2, the existing bionic code
will fall back to performing its own gethostbyaddr call.

Bug: 4344448
Change-Id: I2d03bfec0093c67e8052717c0f499f8871bcfb85
/system/netd/DnsProxyListener.cpp
34ffd85aed3838d53bd1136c751a7825de1940e0 17-Mar-2011 Brad Fitzpatrick <bradfitz@android.com> Reference count SocketClients.

Avoids a netd crash when the client closes its connection while
a DNS request is in-flight.

Depends on Icd7f5f03 in system/core.

Bug: 3438459
Change-Id: Ie6953196623b97ad2b90df951186fbfce84f8e4e
/system/netd/DnsProxyListener.cpp
0a60eb99298e54a3fca719f6695b1d7347dc849a 22-Feb-2011 David Turner <digit@android.com> am 54a9575b: Merge "Add GetHostByAddrHandler to DnsProxyListener"

* commit '54a9575bb7a83ef859ebf4fa2cd610e65b6e9db2':
Add GetHostByAddrHandler to DnsProxyListener
1dbd6cf148ea3fab57ec0644c336e94c78a488be 15-Feb-2011 Mattias Falk <mattias.falk@sonyericsson.com> Add GetHostByAddrHandler to DnsProxyListener

Added functionality to handle gethostbyaddr
in DnsProxyListener. The functionality is
used by getnameinfo. The patch is part of the
work to have one dns cache for all process in
the system.

Change-Id: Ia7609160d2bd1d3bbe2b79956bd208d137a8a9ee
/system/netd/DnsProxyListener.cpp
df9a6a82700a14169cc9d0b74833f1e261b171c6 02-Nov-2010 Brad Fitzpatrick <bradfitz@android.com> Fix the DNS proxy again, for reals.

me == the suck this morning

Bug: 3157030
Change-Id: Ib87ed047cd6999ee3d29b392acc7a559d9bb7349
/system/netd/DnsProxyListener.cpp
89b99f2375271f2020dbb41ae96afd8c17b8e11a 02-Nov-2010 Brad Fitzpatrick <bradfitz@android.com> Fix DNS proxy.

Don't assume writing 0 bytes to SocketClient will be successful. (as
implemented in libsysutils right now, it returns an error...)

Bug: 3157030
Change-Id: I2b2cb8700cbf69be6dffa2e51d1c876c40675b9b
/system/netd/DnsProxyListener.cpp
4e248e5a7a542067fd3f0467680eef2f92b1e195 27-Oct-2010 Brad Fitzpatrick <bradfitz@android.com> DNS proxy thread in netd.

New thread in the netd process which listens on the
/dev/socket/dnsproxyd socket (group owned by 'inet', so only useable
by apps with the INTERNET permission), and does getaddrinfo requests
for other processes, caching in one place, rather than per-app.

Still remaining: proper caching based on DNS-requested lengths, upping
the cache size, getnameinfo, stats, flush, etc, etc.

Change-Id: I1d65af7d87876e508c718656bd81217cd961e20a
/system/netd/DnsProxyListener.cpp
ff2c0d8c13457e43f0d4bf06d3177271aac104c1 17-Nov-2010 Olivier Bailly <olivier@google.com> Add missing include headers for compilation on x86 targets.

Change-Id: I99f7b79bfb5b6305a0772f418a54ace50cac1bbe
/system/netd/DnsProxyListener.cpp
6c798fbf4e89c4058f046a65ee4f9aa5647e43e1 02-Nov-2010 Brad Fitzpatrick <bradfitz@android.com> Fix the DNS proxy again, for reals.

me == the suck this morning

Bug: 3157030
Change-Id: Ib87ed047cd6999ee3d29b392acc7a559d9bb7349
/system/netd/DnsProxyListener.cpp
e6a3a58d6aa4b551e38e1e19fdfea2c0d2841b85 02-Nov-2010 Brad Fitzpatrick <bradfitz@android.com> Fix DNS proxy.

Don't assume writing 0 bytes to SocketClient will be successful. (as
implemented in libsysutils right now, it returns an error...)

Bug: 3157030
Change-Id: I2b2cb8700cbf69be6dffa2e51d1c876c40675b9b
/system/netd/DnsProxyListener.cpp
007e987fee7e815e0c4bc820f434a632b7a69a9d 27-Oct-2010 Brad Fitzpatrick <bradfitz@android.com> DNS proxy thread in netd.

New thread in the netd process which listens on the
/dev/socket/dnsproxyd socket (group owned by 'inet', so only useable
by apps with the INTERNET permission), and does getaddrinfo requests
for other processes, caching in one place, rather than per-app.

Still remaining: proper caching based on DNS-requested lengths, upping
the cache size, getnameinfo, stats, flush, etc, etc.

Change-Id: I1d65af7d87876e508c718656bd81217cd961e20a
/system/netd/DnsProxyListener.cpp