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

Commit 913e7597 authored by Akshata Kadam's avatar Akshata Kadam
Browse files

libmediautils_fuzzer_service_utilities : Resolve timeout 

Resolved a timeout issue that was coming from dprintf().
The value of 'spaces' was very large (int Max value)
and took time to write into a file in MediaPackageManager::dump()
which caused a timeout. Send the 'spaces' value in the
Min/Max FDP range to fix this.

exec/s : 60
Test: ./libmediautils_fuzzer_service_utilities clusterfuzz-
testcase-minimized-libmediautils_fuzzer_service_utilities
-5505455066316800
Bug: 258773755

Change-Id: I82ff50ff95dec6b2ee0429db522caf3fb7c599d5
parent 520c3a58

media/utils/fuzzers/ServiceUtilitiesFuzz.cpp

+4 −1
Original line number Diff line number Diff line
@@ -25,6 +25,7 @@ 


static constexpr int kMaxOperations = 50;

static constexpr int kMaxStringLen = 256;

static constexpr int kMaxSpaces = 1000;



using android::content::AttributionSourceState;
@@ -35,7 +36,9 @@ const std::vector<std::function<void(FuzzedDataProvider*, android::MediaPackageM 
            pm.allowPlaybackCapture(uid);

        },

        [](FuzzedDataProvider* data_provider, android::MediaPackageManager pm) -> void {

            int spaces = data_provider->ConsumeIntegral<int>();

           /* The large value of spaces was taking time in file write operation.

            * Limited spaces values in range to avoid timeout.*/

            int spaces = data_provider->ConsumeIntegralInRange<int>(0, kMaxSpaces);



            // Dump everything into /dev/null

            int fd = open("/dev/null", O_WRONLY);