From 4b375736d444d4ee95b623e025378530455087aa Mon Sep 17 00:00:00 2001 From: Aleksandr Nogikh Date: Fri, 18 Jul 2025 15:23:19 +0200 Subject: pkg/manager: don't check coverage filters too early Right at the Loop() start, these are not filled out yet. --- pkg/manager/diff.go | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'pkg') diff --git a/pkg/manager/diff.go b/pkg/manager/diff.go index aa640515d..3f4a0fd34 100644 --- a/pkg/manager/diff.go +++ b/pkg/manager/diff.go @@ -271,6 +271,15 @@ func (dc *diffContext) monitorPatchedCoverage(ctx context.Context) error { // The feature is disabled. return nil } + + // First wait until we have almost triaged all of the corpus. + select { + case <-ctx.Done(): + return nil + case <-dc.waitCorpusTriage(ctx, corpusTriageToMonitor): + } + + // By this moment, we must have coverage filters already filled out. focusPCs := 0 // The last one is "everything else", so it's not of interest. coverFilters := dc.new.coverFilters @@ -283,13 +292,6 @@ func (dc *diffContext) monitorPatchedCoverage(ctx context.Context) error { return nil } - // First wait until we have almost triaged all of the corpus. - select { - case <-ctx.Done(): - return nil - case <-dc.waitCorpusTriage(ctx, corpusTriageToMonitor): - } - // Then give the fuzzer some change to get through. select { case <-time.After(dc.cfg.FuzzToReachPatched): -- cgit mrf-deployment