commit e61c9b10cb868e6148b6e7271b856e948241705d
author Todd Kjos <tkjos@google.com> Tue Oct 03 14:04:39 2017 -0700
committer Todd Kjos <tkjos@google.com> Tue Oct 03 15:53:51 2017 -0700
tree 6b3b07fdfbb42702069bee61286afef248396490
parent 42ec4974152046ebffef7e0e31015fa0bc4e9a7b

Fix off-by-one error with binderThroughputTest -s

The -s option to binderThroughputTest allows the user
to set the size of the payload being sent during the
test. Due to an off-by-one error, the size was actually
4 bytes less than specified. Fixed.

Test: verified correct size
Bug: 65052700
Change-Id: I884fe08c113c2ca63a6f328741fb0914799e9eb7
Signed-off-by: Todd Kjos <tkjos@google.com>

libs/binder/tests/binderThroughputTest.cpp

diff --git a/libs/binder/tests/binderThroughputTest.cpp b/libs/binder/tests/binderThroughputTest.cpp
index e35f491..bf41e0b 100644
--- a/libs/binder/tests/binderThroughputTest.cpp
+++ b/libs/binder/tests/binderThroughputTest.cpp
@@ -215,7 +215,7 @@
         int target = cs_pair ? num % server_count : rand() % workers.size();
         int sz = payload_size;
 
-        while (sz > sizeof(uint32_t)) {
+        while (sz >= sizeof(uint32_t)) {
             data.writeInt32(0);
             sz -= sizeof(uint32_t);
         }