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.15.x (#18564) 

* backport of commit 06d4c72f68

* backport of commit 42c37bb2c1

---------

Co-authored-by: cskh <hui.kang@hashicorp.com>
pull/18569/head
hc-github-team-consul-core 2023-08-23 09:59:25 -05:00 committed by GitHub
parent 5c46e020b2
commit d738be6aa8
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

@ -834,6 +834,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