diff options
| author | Elliott Hughes <enh@google.com> | 2016-08-23 23:03:20 +0000 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2016-08-23 23:03:20 +0000 |
| commit | 8615d79d85e53c86028b702152bc00864b18f9d5 (patch) | |
| tree | 65afa58c78ea612d8824729717ce18654b4a10ff /libsysutils | |
| parent | 504d4ccd7cc27806781af880cb2479075c11ad30 (diff) | |
| parent | 299d64144bdf28b06489c64aa4e99c423b84c1c4 (diff) | |
| download | system_core-8615d79d85e53c86028b702152bc00864b18f9d5.tar.gz system_core-8615d79d85e53c86028b702152bc00864b18f9d5.tar.bz2 system_core-8615d79d85e53c86028b702152bc00864b18f9d5.zip | |
Merge "Most accept/accept4 calls in system/core don't actually want the remote address."
am: 299d64144b
Change-Id: I3fb9678dfed08f0b1d6e90e547ecae6cdcabc9a1
Diffstat (limited to 'libsysutils')
| -rw-r--r-- | libsysutils/src/SocketListener.cpp | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/libsysutils/src/SocketListener.cpp b/libsysutils/src/SocketListener.cpp index 4d602a6aa..608abaea5 100644 --- a/libsysutils/src/SocketListener.cpp +++ b/libsysutils/src/SocketListener.cpp @@ -199,16 +199,7 @@ void SocketListener::runListener() { continue; } if (mListen && FD_ISSET(mSock, &read_fds)) { - sockaddr_storage ss; - sockaddr* addrp = reinterpret_cast<sockaddr*>(&ss); - socklen_t alen; - int c; - - do { - alen = sizeof(ss); - c = accept4(mSock, addrp, &alen, SOCK_CLOEXEC); - SLOGV("%s got %d from accept", mSocketName, c); - } while (c < 0 && errno == EINTR); + int c = TEMP_FAILURE_RETRY(accept4(mSock, nullptr, nullptr, SOCK_CLOEXEC)); if (c < 0) { SLOGE("accept failed (%s)", strerror(errno)); sleep(1); |