self-hosted/minio
self-hosted/minio
1
0
Fork 0
mirror of https://github.com/minio/minio synced 2024-10-14 20:04:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

tier: avoid stats infinite loop in forwardTo method (#15640)

Browse source 
under some sequence of events following code would
reach an infinite loop.

```
idx1, idx2 := 0, 1
for ; idx2 != idx1; idx2++ {
        fmt.Println(idx2)
}
```

fixes #15639
This commit is contained in:
Harshavardhana 2022-09-01 13:51:06 -07:00 committed by GitHub
parent 5ce1448049
commit f649968c69
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
cmd/tier-last-day-stats.go
View file

@ -38,6 +38,10 @@ func (l *lastDayTierStats) addStats(ts tierStats) {
// forwardTo moves time to t, clearing entries between last update and t.
func (l *lastDayTierStats) forwardTo(t time.Time) {
if t.IsZero() {
t = time.Now()
}
since := t.Sub(l.UpdatedAt).Hours()
// within the hour since l.UpdatedAt
if since < 1 {
@ -45,15 +49,17 @@ func (l *lastDayTierStats) forwardTo(t time.Time) {
}
idx, lastIdx := t.Hour(), l.UpdatedAt.Hour()
l.UpdatedAt = t
l.UpdatedAt = t // update to the latest time index
if since >= 24 {
l.Bins = [24]tierStats{}
return
}
for ; lastIdx != idx; lastIdx++ {
l.Bins[(lastIdx+1)%24] = tierStats{}
for lastIdx != idx {
lastIdx = (lastIdx + 1) % 24
l.Bins[lastIdx] = tierStats{}
}
}