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

Commit ff68a53b 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:... 

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

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



Change-Id: I12ed15a0eb16e57047ede98f37661906855acfc9
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 63c3034c ff741c67

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) {