github
/
consul
mirror of https://github.com/hashicorp/consul
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Backport of bug: prevent go routine leakage due to existing DeferCheck into release/1.16.x (#18565) 

* backport of commit 06d4c72f68

* backport of commit 42c37bb2c1

---------

Co-authored-by: cskh <hui.kang@hashicorp.com>
pull/18570/head
hc-github-team-consul-core 2023-08-23 10:04:19 -05:00 committed by GitHub
parent 985b2f533b
commit 0d0c5d07ac
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
.changelog/18558.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:bug
check: prevent go routine leakage when existing Defercheck of same check id is not nil
```

6
agent/local/state.go
View File

@ -838,6 +838,12 @@ func (l *State) setCheckStateLocked(c *CheckState) {
existing := l.checks[id]
if existing != nil {
c.InSync = c.Check.IsSame(existing.Check)
// If the existing check has a Defercheck, it needs to be
// assigned to the new check
if existing.DeferCheck != nil && c.DeferCheck == nil {
c.DeferCheck = existing.DeferCheck
c.InSync = false
}
}
l.checks[id] = c