commit a4ed5674cd4ef9a61bb34711813085d5205cbf31
author Frederick Mayle <fmayle@google.com> Fri Jun 17 22:03:38 2022 +0000
committer Frederick Mayle <fmayle@google.com> Fri Jun 17 22:09:03 2022 +0000
tree d8deaf5c4ca31b4da3fdd004089b5b0151942d01
parent d05c0b93659d3880a6c4411e1352c9f1d750f2f1

libbinder: Use correct data size in appendFrom test

I noticed this because I had temporarily changed the order of the checks
for a WIP CL and this started failed in the wrong way.

There is a fuzzer for appendFrom, so this shouldn't be a loss of
coverage for the bad data size case.

Test: binderRpcTest
Change-Id: Ia86681c59b2f814f93298d2837d8dcb03c84bdb0

libs/binder/tests/binderRpcTest.cpp

diff --git a/libs/binder/tests/binderRpcTest.cpp b/libs/binder/tests/binderRpcTest.cpp
index 0247e42..4977c63 100644
--- a/libs/binder/tests/binderRpcTest.cpp
+++ b/libs/binder/tests/binderRpcTest.cpp
@@ -755,7 +755,7 @@
     p1.markForBinder(proc1.rootBinder);
     p1.writeInt32(3);
 
-    EXPECT_EQ(BAD_TYPE, p1.appendFrom(&pRaw, 0, p1.dataSize()));
+    EXPECT_EQ(BAD_TYPE, p1.appendFrom(&pRaw, 0, pRaw.dataSize()));
     EXPECT_EQ(BAD_TYPE, pRaw.appendFrom(&p1, 0, p1.dataSize()));
 
     Parcel p2;