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

Commit 4e99e1fa authored by Rachel Lee's avatar Rachel Lee
Browse files

Fix fuzzer: VSyncPredictor::mIdealPeriod=0 

Change fuzzer so it does not pass 0 for mIdealPeriod to avoid div by 0.

Bug: 289711305
Bug: 242440371
Test: atest VSyncPredictorTest
Change-Id: I4dd7bf7a96c50b98a3c5fa0a417dea5d63afbf6d
parent d671d264

services/surfaceflinger/fuzzer/surfaceflinger_scheduler_fuzzer.cpp

+3 −2
Original line number Diff line number Diff line
@@ -177,8 +177,9 @@ void SchedulerFuzzer::fuzzVSyncPredictor() { 
    uint16_t now = mFdp.ConsumeIntegral<uint16_t>();

    uint16_t historySize = mFdp.ConsumeIntegralInRange<uint16_t>(1, UINT16_MAX);

    uint16_t minimumSamplesForPrediction = mFdp.ConsumeIntegralInRange<uint16_t>(1, UINT16_MAX);

    scheduler::VSyncPredictor tracker{kDisplayId, mFdp.ConsumeIntegral<uint16_t>() /*period*/,

                                      historySize, minimumSamplesForPrediction,

    nsecs_t idealPeriod = mFdp.ConsumeIntegralInRange<nsecs_t>(1, UINT32_MAX);

    scheduler::VSyncPredictor tracker{kDisplayId, idealPeriod, historySize,

                                      minimumSamplesForPrediction,

                                      mFdp.ConsumeIntegral<uint32_t>() /*outlierTolerancePercent*/};

    uint16_t period = mFdp.ConsumeIntegral<uint16_t>();

    tracker.setPeriod(period);