sendnrw/windows_exporter
2
0
Fork 0
mirror of https://github.com/prometheus-community/windows_exporter.git synced 2026-03-04 17:46:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #840 from newrelic-forks/fix_service_memory_leak

Browse Source 
Service Api collection close servicehandler to avoid memory leak
This commit is contained in:
Ben Reedy
2021-09-25 18:22:21 +10:00
committed by GitHub
parent 5931604b58 f4f5aaf146
commit 3e37b7b6f0
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
collector/service.go
View File

@@ -246,18 +246,17 @@ func (c *serviceCollector) collectAPI(ch chan<- prometheus.Metric) error {
if err != nil { if err != nil {
continue continue
} }
defer serviceHandle.Close()
// Get Service Configuration // Get Service Configuration
serviceConfig, err := serviceHandle.Config() serviceConfig, err := serviceHandle.Config()
if err != nil { if err != nil {
_ = serviceHandle.Close()
continue continue
} }
// Get Service Current Status // Get Service Current Status
serviceStatus, err := serviceHandle.Query() serviceStatus, err := serviceHandle.Query()
if err != nil { if err != nil {
_ = serviceHandle.Close()
continue continue
} }