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

Commit 4f65172f authored by Frederick Mayle's avatar Frederick Mayle
Browse files

libbinder: avoid uninteresting protocol version crash in fuzzer 

Bug: 237640108
Test: android.hardware.vibrator-service.example_fuzzer
Change-Id: I6c4d2b19c541a85a3236f0133d3dd0c0575ab444
parent 23f910cb

libs/binder/tests/parcel_fuzzer/random_parcel.cpp

+5 −0
Original line number Diff line number Diff line
@@ -45,6 +45,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) {