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

Commit ff741c67 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "libbinder: avoid uninteresting protocol version crash in fuzzer" am: fafdb185 

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



Change-Id: Ib728f9fe814944db0fe30f21ceb561194abc6334
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 21951c8e fafdb185

libs/binder/tests/parcel_fuzzer/random_parcel.cpp

+5 −0
Original line number Diff line number Diff line
@@ -36,6 +36,11 @@ void fillRandomParcel(Parcel* p, FuzzedDataProvider&& provider, RandomParcelOpti 
    if (provider.ConsumeBool()) {

        auto session = RpcSession::make(RpcTransportCtxFactoryRaw::make());

        CHECK_EQ(OK, session->addNullDebuggingClient());

        // Set the protocol version so that we don't crash if the session

        // actually gets used. This isn't cheating because the version should

        // always be set if the session init succeeded and we aren't testing the

        // session init here (it is bypassed by addNullDebuggingClient).

        session->setProtocolVersion(RPC_WIRE_PROTOCOL_VERSION);

        p->markForRpc(session);



        if (options->writeHeader) {