diff options
author | Michael Lentine <mlentine@google.com> | 2014-10-02 09:11:04 -0700 |
---|---|---|
committer | Jesse Hall <jessehall@google.com> | 2014-10-02 18:13:02 +0000 |
commit | 36273c991b189c5516819f4d5e3f83061c320593 (patch) | |
tree | bdad95ad79cc4f077fc773536084ead9dfb8b41f /libs/binder/Parcel.cpp | |
parent | 5ee2c9d20c0fbecf6b4a482eb8d8bfdf85d0d424 (diff) | |
download | frameworks_native-36273c991b189c5516819f4d5e3f83061c320593.zip frameworks_native-36273c991b189c5516819f4d5e3f83061c320593.tar.gz frameworks_native-36273c991b189c5516819f4d5e3f83061c320593.tar.bz2 |
Add more logging for dup(fd) failure
Bug: 17477219
Change-Id: Ide0ae16d777c9af783023c705c18a93c00999147
Diffstat (limited to 'libs/binder/Parcel.cpp')
-rw-r--r-- | libs/binder/Parcel.cpp | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/libs/binder/Parcel.cpp b/libs/binder/Parcel.cpp index 76504ce..ba5688c 100644 --- a/libs/binder/Parcel.cpp +++ b/libs/binder/Parcel.cpp @@ -36,6 +36,7 @@ #include <private/binder/binder_module.h> +#include <fcntl.h> #include <inttypes.h> #include <stdio.h> #include <stdlib.h> @@ -1280,11 +1281,22 @@ status_t Parcel::read(FlattenableHelperInterface& val) const status_t err = NO_ERROR; for (size_t i=0 ; i<fd_count && err==NO_ERROR ; i++) { - fds[i] = dup(this->readFileDescriptor()); + int oldfd = this->readFileDescriptor(); + fds[i] = dup(oldfd); if (fds[i] < 0) { + int dupErrno = errno; err = BAD_VALUE; - ALOGE("dup() failed in Parcel::read, i is %zu, fds[i] is %d, fd_count is %zu, error: %s", - i, fds[i], fd_count, strerror(errno)); + int flags = fcntl(oldfd, F_GETFD); + int fcntlErrno = errno; + const flat_binder_object* flat = readObject(true); + ALOGE("dup failed in Parcel::read, fd %zu of %zu\n" + " dup(%d) = %d [errno: %d (%s)]\n" + " fcntl(%d, F_GETFD) = %d [errno: %d (%s)]\n" + " flat %p type %d", + i, fd_count, + oldfd, fds[i], dupErrno, strerror(dupErrno), + oldfd, flags, fcntlErrno, strerror(fcntlErrno), + flat, flat ? flat->type : 0); } } |