Fix leaked healing goroutines (#14322)

Only the first `listAndHeal` would ever be able to write on errCh, blocking all others infinitely.

Instead read all errors but return the first non-nil, if any.

The intention appears to be that this should cancel on any error, 
so that part is kept. 

Regression from #13990
This commit is contained in:
Klaus Post 2022-02-16 08:40:18 -08:00 committed by GitHub
parent 27d94c64ed
commit 60cd513a33
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -1848,7 +1848,15 @@ func (z *erasureServerPools) HealObjects(ctx context.Context, bucket, prefix str
}
wg.Wait()
}()
return <-errCh
var err error
for e := range errCh {
// Save first non-nil error.
if e != nil && err != nil {
err = e
cancel()
}
}
return err
}
func (z *erasureServerPools) HealObject(ctx context.Context, bucket, object, versionID string, opts madmin.HealOpts) (madmin.HealResultItem, error) {