diff options
| author | 2023-10-09 11:22:40 -0700 | |
|---|---|---|
| committer | 2023-10-09 11:29:45 -0700 | |
| commit | 4e99e1fa7034f628c718c1fce1dd4837d9f14464 (patch) | |
| tree | db682fa5445aab069b85bfa1aaed6629a699586e | |
| parent | d671d2642c3d44240d541726bb41fa549c3c5028 (diff) | |
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
| -rw-r--r-- | services/surfaceflinger/fuzzer/surfaceflinger_scheduler_fuzzer.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/services/surfaceflinger/fuzzer/surfaceflinger_scheduler_fuzzer.cpp b/services/surfaceflinger/fuzzer/surfaceflinger_scheduler_fuzzer.cpp index cbbfa1695c..be2a22a121 100644 --- a/services/surfaceflinger/fuzzer/surfaceflinger_scheduler_fuzzer.cpp +++ b/services/surfaceflinger/fuzzer/surfaceflinger_scheduler_fuzzer.cpp @@ -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); |