Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 60c292a5 authored by Steven Moreland's avatar Steven Moreland Committed by Automerger Merge Worker
Browse files

Merge "binder_rpc_fuzzer: limit connections" am: 13934429 

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2368629



Change-Id: I8f93359e431169a8883bf4e7afba35beae45867e
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 38052352 13934429

libs/binder/tests/rpc_fuzzer/main.cpp

+6 −1
Original line number Diff line number Diff line
@@ -133,8 +133,13 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { 


    bool hangupBeforeShutdown = provider.ConsumeBool();



    // b/260736889 - limit arbitrarily, due to thread resource exhaustion, which currently

    // aborts. Servers should consider RpcServer::setConnectionFilter instead.

    constexpr size_t kMaxConnections = 1000;



    while (provider.remaining_bytes() > 0) {

        if (connections.empty() || provider.ConsumeBool()) {

        if (connections.empty() ||

            (connections.size() < kMaxConnections && provider.ConsumeBool())) {

            base::unique_fd fd(TEMP_FAILURE_RETRY(socket(AF_UNIX, SOCK_STREAM | SOCK_CLOEXEC, 0)));

            CHECK_NE(fd.get(), -1);

            CHECK_EQ(0,