summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Rachel Lee <rnlee@google.com> 2023-10-09 11:22:40 -0700
committer Rachel Lee <rnlee@google.com> 2023-10-09 11:29:45 -0700
commit4e99e1fa7034f628c718c1fce1dd4837d9f14464 (patch)
treedb682fa5445aab069b85bfa1aaed6629a699586e
parentd671d2642c3d44240d541726bb41fa549c3c5028 (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.cpp5
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);