commit 57b1591dca003b488d42e85dfdc5e096d35fdf8e
author Steven Moreland <smoreland@google.com> Fri Jun 02 21:35:00 2023 +0000
committer Steven Moreland <smoreland@google.com> Fri Jun 02 21:35:00 2023 +0000
tree 374efa0a33be87f2523e5eb52040cdf1bfe30103
parent 4afa2c395765c7d944505c94f46c4172f6476975

libbinder_driver: never enforce data avail

It would be good to cover this too, but we do have it covered
via binder_parcel_fuzzer now, and this causes a lot of wasted
cycles for our AIDL fuzzers.

Bug: N/A
Test: diffing strace of runs before/after
Change-Id: I743b62272c402c400c5e357adf5538dc5440f2a6

libs/binder/tests/parcel_fuzzer/libbinder_driver.cpp

diff --git a/libs/binder/tests/parcel_fuzzer/libbinder_driver.cpp b/libs/binder/tests/parcel_fuzzer/libbinder_driver.cpp
index 0b3902d..cb2878c 100644
--- a/libs/binder/tests/parcel_fuzzer/libbinder_driver.cpp
+++ b/libs/binder/tests/parcel_fuzzer/libbinder_driver.cpp
@@ -52,7 +52,7 @@
                     uint32_t flags = provider.ConsumeIntegral<uint32_t>();
                     Parcel data;
                     // for increased fuzz coverage
-                    data.setEnforceNoDataAvail(provider.ConsumeBool());
+                    data.setEnforceNoDataAvail(false);
 
                     sp<IBinder> target = options.extraBinders.at(
                             provider.ConsumeIntegralInRange<size_t>(0,
@@ -73,7 +73,7 @@
 
                     Parcel reply;
                     // for increased fuzz coverage
-                    reply.setEnforceNoDataAvail(provider.ConsumeBool());
+                    reply.setEnforceNoDataAvail(false);
                     (void)target->transact(code, data, &reply, flags);
 
                     // feed back in binders and fds that are returned from the service, so that