diff options
| author | Ben Cheng <bccheng@google.com> | 2011-12-01 17:11:32 -0800 |
|---|---|---|
| committer | Ben Cheng <bccheng@google.com> | 2011-12-01 17:11:32 -0800 |
| commit | d640f89205e17d9f19b11c2954862f0945a40e7b (patch) | |
| tree | 48623910fe75822605f4a4c5884e8b8bcd745d86 | |
| parent | ea402a7e8c421b2e2403cfe9d8575c8047b315b7 (diff) | |
| download | frameworks_native-d640f89205e17d9f19b11c2954862f0945a40e7b.zip frameworks_native-d640f89205e17d9f19b11c2954862f0945a40e7b.tar.gz frameworks_native-d640f89205e17d9f19b11c2954862f0945a40e7b.tar.bz2 | |
Initialize bwr.read_buffer in IPCThreadState::talkWithDriver.
I/valgrind( 1309): ==1310== Syscall param
ioctl(BINDER_WRITE_READ).read_buffer points to uninitialised byte(s)
I/valgrind( 1309): ==1310== at 0x480E670: __ioctl (__ioctl.S:10)
I/valgrind( 1309): ==1310== Address 0xbda651b4 is on thread 1's stack
Change-Id: I02893df7b5786b6b2dbd9659f5706d7171295ab2
| -rw-r--r-- | libs/binder/IPCThreadState.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/libs/binder/IPCThreadState.cpp b/libs/binder/IPCThreadState.cpp index 5ccf87f..5d34787 100644 --- a/libs/binder/IPCThreadState.cpp +++ b/libs/binder/IPCThreadState.cpp @@ -773,6 +773,7 @@ status_t IPCThreadState::talkWithDriver(bool doReceive) bwr.read_buffer = (long unsigned int)mIn.data(); } else { bwr.read_size = 0; + bwr.read_buffer = 0; } IF_LOG_COMMANDS() { |